A comma can be added to the next "If not further configured deletes data in columns that were removed or had their type changed in the model."
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 45/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Stale
- Tech stack
- html
- Domain
- documentation
Research direction
Start at the linked Method sync API reference and locate the description for the options.alter.drop parameter. Add the introductory comma to the sentence, then verify that the updated wording appears correctly in the website documentation.
Written by the indexing model from the issue text.
Description
Issue Description
I found a possibly grammar improvement for the description of one of the sync function's parameters: options.alter.drop
<--
Try to be as clear as possible. Don't assume that the maintainers will immediately understand the problem. Write your issue in a way that new contributors can also help (add links to helpful resources when applicable).
-->
What was unclear/insufficient/not covered in the documentation
The sentence is clear, no error found, just a suggestion.
If possible: Provide some suggestion on how we can enhance the docs
Add an introductory comma to the aforementioned text, to connect better the clauses:
"If not further configured*,* deletes data in columns that were removed or had their type changed in the model."
Additional context
<-- Add any other context or screenshots about the issue here. -->
- Dominant language
- HTML
- Stars
- 30
- Forks
- 170
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from sequelize/website
-
Difficulty 1/5 1-3 hours Newbie friendliness 74/100
-
Update Migration.md Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
All issues in sequelize/website
Similar issues
-
documentation help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
-
user-reported
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Kong/developer.konghq.com#7316 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
HarperFast/skills#96 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
infinispan/infinispan#18150 ·