Skip to content
New issue

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

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

Already on GitHub? # to your account

Log correct response status code #2149

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Log correct response status code #2149

wants to merge 2 commits into from

Conversation

JHvolka
Copy link
Contributor

@JHvolka JHvolka commented Jan 23, 2025

No description provided.

Copy link

github-actions bot commented Jan 23, 2025

Test build pipeline info 🚀

Changes in the code and tag info:

➡️ Changes in forms-shared: false

➡️ Changes in next: false

➡️ Changes in strapi: false

➡️ Changes in nest-forms-backend: true

➡️ Changes in nest-clamav-scanner: false

➡️ Changes in nest-tax-backend: false

➡️ Changes in nest-city-account: true

We are going to build 🚢

🔜 nest-forms-backend

Copy link
Contributor

@mpinter mpinter left a comment

Choose a reason for hiding this comment

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

Please fix linter/validation errors and do the same change for statusMessage as well. Can merge without further review afterwards.

Check the other logger.service as well.

@github-actions github-actions bot added the pr: fix & ship 🚢 No additional review needed before merge - some work may be required, if specified in last review label Jan 23, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
pr: fix & ship 🚢 No additional review needed before merge - some work may be required, if specified in last review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants