Skip to content

Conversation

soyuka
Copy link
Member

@soyuka soyuka commented Jul 29, 2025

Copy link
Contributor

Choose a reason for hiding this comment

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

Don't we need to fix the YamlResourceExtractor too ?

Copy link
Member Author

Choose a reason for hiding this comment

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

probably

Copy link
Contributor

Choose a reason for hiding this comment

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

Are this changes still needed as the MR #7315 has been merged ?

Otherwise, i realized than i have done partially the job (see: #7322).
If you want to integrate this changes here i can decline the MR.

@soyuka soyuka merged commit d1abfc0 into api-platform:4.1 Jul 29, 2025
1 check failed
@soyuka
Copy link
Member Author

soyuka commented Jul 29, 2025

We need this at least for the schema and I think having defaults is still a nice thing to have.

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

Labels

None yet

3 participants