Skip to content

[Uid] Make AbstractUid implement Ds\Hashable if available #19948

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
Closed
fabpot opened this issue Jun 7, 2024 · 2 comments
Closed

[Uid] Make AbstractUid implement Ds\Hashable if available #19948

fabpot opened this issue Jun 7, 2024 · 2 comments
Labels
Uid
Milestone
7.2

Comments

@fabpot
Copy link
Member

fabpot commented Jun 7, 2024

Q A
Feature PR symfony/symfony#57313
PR author(s) @jahudka
Merged in 7.2

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 7.2 branch in the symfony-docs repository.

Thank you! 😃
@fabpot fabpot added this to the 7.2 milestone Jun 7, 2024
@carsonbot carsonbot added the Uid label Jun 7, 2024
@xabbuh
Copy link
Member

xabbuh commented Jun 7, 2024

IMO this is an implementation detail that we do not need to document

@symfony/docs What do you think?

@OskarStark OskarStark closed this as not planned Won't fix, can't repro, duplicate, stale Jun 7, 2024
@wouterj
Copy link
Member

wouterj commented Jun 7, 2024

Agreed, this is a classic "it just works" feature that we don't need to document explicitly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Uid
Projects
None yet
Milestone
7.2
Development

No branches or pull requests
5 participants
@fabpot @wouterj @OskarStark @xabbuh @carsonbot