-
Notifications
You must be signed in to change notification settings - Fork 386
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
docs: add useDefaultIdType and generated to property properties #918
Conversation
@agnes512, i think your commit message is too long (max 50 characters for the first line). Could you please fix it? Thanks. |
<td>No</td> | ||
<td>Boolean</td> | ||
<td> | ||
<strong>For auto-migrate usage and the memory connector(id property only)</strong>. When it sets to <code>true</code>, the value of the property will be generated automatically with the default property type of the database. (e.g integer for MySQL and string for MongoDB) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not knowing the actual behavior, I'm a bit confused on this line:
For auto-migrate usage and the memory connector(id property only)
Does it mean it only works for:
- auto-migrate
- for memory connector, it only works for id property. For other connectors, there's no such restriction?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reworded and removed memory-connector part.
29304fa
to
e7edc3e
Compare
e7edc3e
to
ce5a8bf
Compare
ce5a8bf
to
0bdd337
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Add docs for property's property
useDefaultIdType
added in PR.Also add explanation for
generated
to help explainuseDefaultIdType
.