Skip to content

Reintrospect uuid() and cuid()#1030

Merged
do4gr merged 1 commit intomasterfrom
ReIntrospectIdDefaults
Aug 17, 2020
Merged

Reintrospect uuid() and cuid()#1030
do4gr merged 1 commit intomasterfrom
ReIntrospectIdDefaults

Conversation

@do4gr
Copy link
Copy Markdown
Member

@do4gr do4gr commented Aug 14, 2020

@do4gr do4gr marked this pull request as draft August 14, 2020 18:37
@do4gr do4gr added this to the Milestone 2.5.0 milestone Aug 14, 2020
@do4gr do4gr mentioned this pull request Aug 15, 2020
Closed
@do4gr do4gr marked this pull request as ready for review August 15, 2020 10:05
tomhoule
tomhoule approved these changes Aug 17, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@tomhoule tomhoule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@do4gr do4gr merged commit c3b91c8 into master Aug 17, 2020
@do4gr do4gr deleted the ReIntrospectIdDefaults branch August 17, 2020 08:03
@Jolg42 Jolg42 mentioned this pull request Aug 27, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@tomhoule tomhoule tomhoule approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

Milestone 2.5.0

Development

Successfully merging this pull request may close these issues.

[Re-Introspection] Keep defaults that are not persisted in the database (cuid() and uuid())

2 participants

@do4gr @tomhoule