Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0] [IMP] rest_log: add MissingError to dispatch method #412

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

renda-dev
Copy link
Contributor

@renda-dev renda-dev commented Feb 14, 2024

Add RESTServiceMissingErrorException to the exceptions to have the right status code returned when a MissingError is raised and intercepted

@OCA-git-bot
Copy link
Contributor

Hi @simahawk,
some modules you are maintaining are being modified, check this out!

@renda-dev renda-dev force-pushed the 14.0-imp-rest_log-dispatch branch from e9c6637 to e591bd9 Compare February 14, 2024 12:02
Copy link

@aleuffre aleuffre left a comment

Choose a reason for hiding this comment

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

Code review

@renda-dev renda-dev force-pushed the 14.0-imp-rest_log-dispatch branch from e591bd9 to 8c3e05c Compare March 12, 2024 08:15
@lmignon
Copy link
Contributor

lmignon commented Apr 16, 2024

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-412-by-lmignon-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit dba9795 into OCA:14.0 Apr 16, 2024
7 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e8bd68a. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants