Skip to content

Conversation

@BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Apr 12, 2025

No description provided.

renovate bot and others added 30 commits March 19, 2025 22:39
chore(deps): update amazon/aws-cli docker tag to v2.24.27
fix(deps): update sentry-javascript monorepo to v9.7.0
chore(deps): update amazon/aws-cli docker tag to v2.25.0
chore(deps): update varnish docker tag to v7.7.0
fix(deps): update dependency axios to v1.8.4
chore(deps): update dependency @types/node to v22.13.11
fix(deps): update dependency @pulumi/aws to v6.73.0
chore(deps): update amazon/aws-cli docker tag to v2.25.1
chore(deps): update dependency @eslint/eslintrc to v3.3.1
chore(deps): update eslint monorepo to v9.23.0
It was a little tricky, but finally i figured out how to do it:
https://api-platform.com/docs/core/errors/

The big impact is, that the serialization of the errors is now
done with the same infrastructure as serializing entities.
That makes it less format dependent.

It seems a little weird that we use a provider to "replace" the exception path
of the response by extracting the exception to the context,
and then continuing like when we would serialize a response.
But i can also see the benefits.
See api-platform/core#5974

Issue: #6618
@BacLuc BacLuc requested review from a team, manuelmeister, pmattmann, simfeld and usu April 12, 2025 11:55
@BacLuc BacLuc temporarily deployed to staging April 12, 2025 13:49 — with GitHub Actions Inactive
@carlobeltrame carlobeltrame temporarily deployed to staging April 12, 2025 13:49 Destroyed
Copy link
Member

@manuelmeister manuelmeister left a comment

Choose a reason for hiding this comment

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

Checklist overview is not loading anymore

@BacLuc BacLuc temporarily deployed to staging April 12, 2025 14:06 — with GitHub Actions Inactive
@carlobeltrame carlobeltrame temporarily deployed to staging April 12, 2025 14:06 Destroyed
@BacLuc BacLuc requested a review from manuelmeister April 12, 2025 14:47
@BacLuc
Copy link
Contributor Author

BacLuc commented Apr 12, 2025

Checklist overview is not loading anymore

is only on the pull request

@BacLuc
Copy link
Contributor Author

BacLuc commented Apr 12, 2025

/fast-forward

@BacLuc BacLuc merged commit e273e06 into prod Apr 12, 2025
164 of 167 checks passed
@BacLuc BacLuc temporarily deployed to prod April 12, 2025 15:16 — with GitHub Actions Inactive
@carlobeltrame carlobeltrame temporarily deployed to prod April 12, 2025 15:16 Destroyed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants