Skip to content

[18.0][ADD] fastapi_log#584

Merged
OCA-git-bot merged 17 commits into
OCA:18.0from
akretion:18.0-add-fastapi_log
May 27, 2026
Merged

[18.0][ADD] fastapi_log#584
OCA-git-bot merged 17 commits into
OCA:18.0from
akretion:18.0-add-fastapi_log

Conversation

@paradoxxxzero

@paradoxxxzero paradoxxxzero commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

Migration to 18.0 of #554

paradoxxxzero and others added 16 commits January 14, 2026 17:02
@paradoxxxzero
[ADD] fastapi_log
973c5ef
@SirPyTech @paradoxxxzero
[REF] fastapi_log: Extract common features to api_log
f9692bb 
This way other APIs might use the new module `api_log` to store logs.
@SirPyTech @paradoxxxzero
[ADD] fastapi_log_mail
9fbf940
@SirPyTech @paradoxxxzero
[FIX] fastapi_log: Manage multi-slash endpoints
2394468
@SirPyTech @paradoxxxzero
[IMP] api_log: Hide sensitive headers
0c154fa
@SirPyTech @paradoxxxzero
[IMP] api_log: Add collection of logs
4d10c51
@SirPyTech @paradoxxxzero
[ADD] api_log_mail: Notify user about logged exceptions
a89f4af
@SirPyTech @paradoxxxzero
[IMP] fastapi_log: Adapt to log collection
ce3931f
@SirPyTech @paradoxxxzero
[IMP] fastapi_log_mail: Adapt to log collection
1ecfa12
@SirPyTech @paradoxxxzero
[IMP] api_log: Inject Log record identifier
6ee6d7c
@SirPyTech @paradoxxxzero
[IMP] fastapi_log: Use dedicated cursor for logs
1c22261 
Co-authored-by: Florian Mounier <paradoxxx.zero@gmail.com>
@SirPyTech @paradoxxxzero
[IMP] fastapi_log_mail: Use dedicated cursor in tests
d16d556
@SirPyTech @paradoxxxzero
[IMP] fastapi_log: In tests, check Log identifier in response
5117b58
@paradoxxxzero
[MIG] api_log
d0aeb41
@paradoxxxzero
[MIG] api_log_mail
2c24182
@paradoxxxzero
[MIG] fastapi_log
6447d9b
@paradoxxxzero paradoxxxzero marked this pull request as ready for review January 15, 2026 17:46
SirPyTech
SirPyTech suggested changes Mar 26, 2026
View reviewed changes

@SirPyTech SirPyTech left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#554 has finally been merged 🎉

Please get the latest changes from there and split automatic (pre-commit) changes from manual migration changes as explained in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0#technical-method-to-migrate-a-module-from-170-to-180-branch.

I suggest using https://github.com/OCA/odoo-module-migrator to create the new migration branches and then place your existing migration commits on top of it.

@SirPyTech SirPyTech mentioned this pull request Mar 27, 2026
Merged
@paradoxxxzero

paradoxxxzero commented Mar 31, 2026

Copy link
Copy Markdown
Contributor Author

Yes I know but this PR wasn’t a standard migration but rather a PR port. I’ll update it to be a regular migration now that #554 has been merged if nobody does it before

@lmignon

lmignon commented May 27, 2026

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-584-by-lmignon-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 340b576 into OCA:18.0 May 27, 2026
7 checks passed
@OCA-git-bot

OCA-git-bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at e36297a. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@SirPyTech SirPyTech SirPyTech requested changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

merged 🎉

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@paradoxxxzero @lmignon @OCA-git-bot @SirPyTech