Skip to content

Conversation

norkunas
Copy link
Contributor

@norkunas norkunas commented Mar 25, 2021

Q A
Branch? 2.6
Bug fix? yes
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Add a PropertySchemaRangeRestriction to transform for integers and floats Range validation constraint into minimum/maximum JSON Schema restriction.

@dunglas
Copy link
Member

dunglas commented Mar 25, 2021

However, this is a new feature. Could you target the main branch instead of 2.6 please?

@norkunas
Copy link
Contributor Author

previous similar PR's were merged as a bugfixes :/

@norkunas norkunas force-pushed the property-schema-range-restriction branch from 81faeb0 to 4f20503 Compare March 25, 2021 09:31
@alanpoulain
Copy link
Member

previous similar PR's were merged as a bugfixes :/

Yes, for reference: #4139 (comment).
Seems OK to me to merge it in 2.6.

Could you add a CHANGELOG entry too?

@norkunas norkunas force-pushed the property-schema-range-restriction branch from 4f20503 to e1d443c Compare March 25, 2021 09:42
@norkunas
Copy link
Contributor Author

previous similar PR's were merged as a bugfixes :/

Yes, for reference: #4139 (comment).
Seems OK to me to merge it in 2.6.

Could you add a CHANGELOG entry too?

Sure, added.

@soyuka soyuka changed the base branch from 2.6 to main March 25, 2021 13:09
@soyuka soyuka force-pushed the property-schema-range-restriction branch from e1d443c to 906b385 Compare March 25, 2021 13:09
@norkunas norkunas force-pushed the property-schema-range-restriction branch from 906b385 to ba3cd8d Compare March 25, 2021 13:28
@norkunas norkunas changed the title Fix generating property schema with Range restriction Add support for generating property schema with Range restriction Mar 25, 2021
@norkunas norkunas force-pushed the property-schema-range-restriction branch from ba3cd8d to 4aa39e1 Compare March 25, 2021 13:46
@norkunas norkunas force-pushed the property-schema-range-restriction branch from 4aa39e1 to 2d84672 Compare March 26, 2021 10:55
@alanpoulain alanpoulain merged commit 359b373 into api-platform:main Mar 26, 2021
@alanpoulain
Copy link
Member

Thank you again @norkunas.

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.

3 participants