Skip to content
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.

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

[Uid] Add UidFactory to create Ulid and Uuid from timestamps and random… #14960

Closed
nicolas-grekas opened this issue Feb 11, 2021 · 7 comments
Closed
Labels
Milestone

Comments

@nicolas-grekas
Copy link
Member

Q A
Feature PR symfony/symfony#39507
PR author(s) @fancyweb
Merged in 5.3-dev
@carsonbot carsonbot added the Uid label Feb 11, 2021
@xabbuh xabbuh added this to the 5.3 milestone Feb 11, 2021
@javiereguiluz
Copy link
Member

In addition to the semantic config of UID defaults, the most important thing to document here are the Doctrine UID generators (we had them in the past, but we removed it and current docs say that there are no generators, which is now wrong).

@parijke
Copy link
Contributor

parijke commented Jun 1, 2021

Documentation says there are no uuid generators, but there is:
use Symfony\Bridge\Doctrine\IdGenerator\UuidGenerator;

@carsonbot
Copy link
Collaborator

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@carsonbot
Copy link
Collaborator

Hello? This issue is about to be closed if nobody replies.

@carsonbot
Copy link
Collaborator

Hey,

I didn't hear anything so I'm going to close it. Feel free to comment if this is still relevant, I can always reopen!

@nicolas-grekas
Copy link
Member Author

Could anyone have a look at this one?

@carsonbot carsonbot removed the Stalled label Jun 30, 2022
javiereguiluz added a commit that referenced this issue Jan 9, 2023
…andre-daubois)

This PR was merged into the 5.4 branch.

Discussion
----------

[Uid] Add `UuidFactory` and `UlidFactory` mentions

Solves #14960

Commits
-------

de628c8 [Uid] Add `UuidFactory` and `UlidFactory` mentions
@alamirault
Copy link
Contributor

PR is merged, this issue can be closed 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants