Skip to content

Error-while-phrasing-decrement-method#263

Open
ajaymrbijak wants to merge 2 commits intosequelize:mainfrom
ajaymrbijak:patch-1
Open

Error-while-phrasing-decrement-method#263
ajaymrbijak wants to merge 2 commits intosequelize:mainfrom
ajaymrbijak:patch-1

Conversation

@ajaymrbijak
Copy link
Copy Markdown

increment is changed to decrement in line number 714

increment is changed to decrement in line number 714
@fzn0x
Copy link
Copy Markdown
Member

fzn0x commented Sep 18, 2022

Nice catch

Copy link
Copy Markdown
Member

@WikiRik WikiRik left a comment

Choose a reason for hiding this comment

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

I believe the current wording is correct. However, it might be useful to add

await User.decrement({age: 5}, { where: { id: 1 } }) // Will decrease age to 5

and update line 708 to mention both increment and decrement

Can you also update this in for the v7 docs? https://github.com/sequelize/website/blob/main/docs/core-concepts/model-querying-basics.md

@ajaymrbijak ajaymrbijak requested a review from a team as a code owner April 11, 2024 20:17
@ajaymrbijak ajaymrbijak requested review from WikiRik and ephys April 11, 2024 20:17
@sequelize-bot sequelize-bot Bot added the conflicted This PR has merge conflicts and will not be present in the list of PRs to review label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicted This PR has merge conflicts and will not be present in the list of PRs to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants