-
Notifications
You must be signed in to change notification settings - Fork 697
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
chore: EXPOSED-642 Support both DSL and DAO transform #2309
chore: EXPOSED-642 Support both DSL and DAO transform #2309
Conversation
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.
I would recommend waiting until after PR #2302 is merged to rebase and resolve conflicts since the entire 'DAO-Field-Transformations.topic' is being moved.
Otherwise, if you want to merge first, please consider syncing with @vnikolova after to rebase that PR before it is merged.
documentation-website/Writerside/topics/DAO-Field-Transformations.topic
Outdated
Show resolved
Hide resolved
documentation-website/Writerside/topics/DAO-Field-Transformations.topic
Outdated
Show resolved
Hide resolved
20b874a
to
0ea4859
Compare
@vnikolova I updated some documentation about Transform API here, I also found that kotlin examples were moved into kotlin files, so I did it for new example also. |
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.
Looks good! Thanks for adding the example. Just a couple minor suggestions in the comments.
documentation-website/Writerside/topics/DAO-Entity-definition.topic
Outdated
Show resolved
Hide resolved
documentation-website/Writerside/topics/DAO-Entity-definition.topic
Outdated
Show resolved
Hide resolved
documentation-website/Writerside/topics/DAO-Entity-definition.topic
Outdated
Show resolved
Hide resolved
The deprecation of the DAO transform API was reverted in Exposed 0.57.0, see [1] and [2]. [1]: https://github.com/JetBrains/Exposed/blob/main/CHANGELOG.md#0570 [2]: JetBrains/Exposed#2309 Signed-off-by: Martin Nonnenmacher <[email protected]>
The deprecation of the DAO transform API was reverted in Exposed 0.57.0, see [1] and [2]. [1]: https://github.com/JetBrains/Exposed/blob/main/CHANGELOG.md#0570 [2]: JetBrains/Exposed#2309 Signed-off-by: Martin Nonnenmacher <[email protected]>
Description
With this PR the DAO transform API will be undeprecated. Both DAO and DSL transform will exist.
Affected databases:
Related Issues
EXPOSED-642 Support both DSL and DAO transform