Skip to content

fix(Identifier): set default name as uuid #1102

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
Open
wants to merge 3 commits into
base: next
Choose a base branch
from
Open

fix(Identifier): set default name as uuid #1102

wants to merge 3 commits into from

Conversation

panquez
Copy link
Member

@panquez panquez commented Mar 7, 2025

What do you think about this change?

@panquez
Copy link
Member Author

panquez commented Mar 10, 2025

@BotellaA I have done the change proposed in your comment. Please could you update the serialize to consider know an optional?

@BotellaA
Copy link
Member

@BotellaA I have done the change proposed in your comment. Please could you update the serialize to consider know an optional?

Done

@panquez panquez requested a review from francoisbonneau March 17, 2025 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BotellaA BotellaA BotellaA approved these changes

@MelchiorSchuh MelchiorSchuh Awaiting requested review from MelchiorSchuh

@BenoitTHEBAULT BenoitTHEBAULT Awaiting requested review from BenoitTHEBAULT

@BenPinet BenPinet Awaiting requested review from BenPinet

@francoisbonneau francoisbonneau Awaiting requested review from francoisbonneau

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@panquez @BotellaA @francoisbonneau