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

fix: disable init content for api controllers #452

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hschoenenberger
Copy link
Contributor

@hschoenenberger hschoenenberger commented Nov 21, 2024

As suggested by the merchant in this issue : ACCOUNT-2729, we override initContent with an empty method to avoid triggering hooks.
As a consequence we also avoid related warnings and invalid response JSON.

Copy link

sonarcloud bot commented Nov 21, 2024

@hschoenenberger hschoenenberger changed the title fix: api init content fix: empty init content for api controllers Nov 21, 2024
@hschoenenberger hschoenenberger changed the title fix: empty init content for api controllers fix: disable init content for api controllers Nov 21, 2024
@hschoenenberger hschoenenberger marked this pull request as ready for review November 21, 2024 15:50
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.

3 participants