Skip to content

[Serializer][Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants #58246

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 16, 2024
Merged

[Serializer][Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants #58246

merged 1 commit into from
Sep 16, 2024

Conversation

alexandre-daubois
Copy link
Member

Q A
Branch? 7.2
Bug fix? no
New feature? yes
Deprecations? no
Issues -
License MIT

Following #58238 (cc @fancyweb), for completeness of this 7.2 feature.

@carsonbot carsonbot added this to the 7.2 milestone Sep 12, 2024
@stof
Copy link
Member

stof commented Sep 13, 2024

The comment on the previous PR also suggested adding the alias constant for the serializer component.

@alexandre-daubois alexandre-daubois changed the title [Uid] Add the Uuid::FORMAT_RFC_9562 constant [Serializer][Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants Sep 13, 2024
@carsonbot carsonbot changed the title [Serializer][Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants [Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants Sep 13, 2024
@alexandre-daubois alexandre-daubois changed the title [Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants [Serializer][Uid] Add the Uuid::FORMAT_RFC_9562 and UidNormalizer::NORMALIZATION_FORMAT_RFC9562 constants Sep 13, 2024
@alexandre-daubois
Copy link
Member Author

That's right, let's take advantage of this PR to introduce it as well

@nicolas-grekas
Copy link
Member

Thank you @alexandre-daubois.

@nicolas-grekas nicolas-grekas merged commit b7572a5 into symfony:7.2 Sep 16, 2024
5 of 10 checks passed
@nicolas-grekas nicolas-grekas mentioned this pull request Sep 16, 2024
Closed
@fabpot fabpot mentioned this pull request Oct 27, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stof stof stof approved these changes

@dunglas dunglas Awaiting requested review from dunglas dunglas is a code owner

Assignees
No one assigned
Labels
Feature Status: Needs Review Uid
Projects
None yet
Milestone
7.2
Development

Successfully merging this pull request may close these issues.
4 participants
@alexandre-daubois @stof @nicolas-grekas @carsonbot