Allow setting default value on UUID columns #339
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
so it is possible to set a default value on UUID columns
|
@HeCorr is attempting to deploy a commit to the dottle's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Honestly speaking, this whole idea of arbitrarily preventing the user from setting the default value on any column type is a bad idea. If their database engine does not support it for whatever reason that's up to them, the design tool shouldn't be restricting that in case it is supported by the engine but someone forgot to enable it in the code (which very much seems to be the case here). |
*something* is setting the global cursor style for :disabled elements to "default" and I don't know what, but it conflicts with the intended cursor style of SemiDesign, so let it be determined by semi classes instead.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
1ilit
approved these changes
Feb 2, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #338.
This PR sets the
noDefaultproperty of UUID types (generic and Postgres) tofalseso that default values can be set on UUID columns:This PR also forces the cursor style of disabled elements to be inherited from SemiDesign's classes:
(SemiDesign documentation page)