Skip to content
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

feat: Add findByIdAndUpdate functions to DAO API #1773

Merged
merged 3 commits into from
Jan 11, 2024

Conversation

reidbuzby
Copy link
Contributor

Currently there are no functions in the DAO api that directly allow updates. This change adds functions that allow direct updates in the EntityClass API. Usage guidelines are explained in the documentation updates.

@e5l e5l self-requested a review January 9, 2024 13:43
@e5l e5l self-assigned this Jan 9, 2024
@e5l e5l requested review from joc-a and bog-walk and removed request for e5l January 10, 2024 11:12
@e5l
Copy link
Member

e5l commented Jan 10, 2024

I've fixed tests and dropped an unsupported operator. @joc-a, @bog-walk, please take a look

Copy link
Member

@bog-walk bog-walk left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I left 3 comments about some changes to the api file, documentation .md, and KDocs.

@bog-walk
Copy link
Member

@e5l Could the title of the PR also be changed? Because it currently sounds like update wasn't supported by DAO at all before.

Maybe something like 'feat: Add findByIdAndUpdate functions to dao API'?

@e5l e5l changed the title feat: add update functions to dao API feat: Add findByIdAndUpdate functions to dao API Jan 11, 2024
@e5l e5l changed the title feat: Add findByIdAndUpdate functions to dao API feat: Add findByIdAndUpdate functions to DAO API Jan 11, 2024
@e5l e5l merged commit 8ac9e05 into JetBrains:main Jan 11, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants