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.

Sign up for GitHub

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

Jump to bottom

Save role weight in flag model #1772

Merged
merged 1 commit into from
Oct 23, 2020
Merged

Save role weight in flag model #1772

merged 1 commit into from
Oct 23, 2020

Conversation

Martii
Copy link
Member

@Martii Martii commented Oct 23, 2020 •
edited
Loading

Applies to #126 and post #641

Performed loose migration by dropping all records and recreating just a few that were actively important.

@Martii
Save role weight in flag model
e908b4e 
* When promoting/demoting a User future flagging/unflagging can wreak havoc on critical *(relative)* flags properties.
* Project bump since requires a bit of migration
* Needs followup testing with OpenUserJS#1109

Applies to OpenUserJS#126 and post OpenUserJS#641
@Martii Martii added bug You've guessed it... this means a bug is reported. security Usually relates to something critical. DB Pertains inclusively to the Database operations. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. stability Important to operations. labels Oct 23, 2020
@Martii Martii merged commit 76b018e into OpenUserJS:master Oct 23, 2020
@Martii Martii deleted the Issue-126saveRoleWeightInFlag branch October 23, 2020 01:53
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Oct 23, 2020
@Martii
Sync devDBdirty
97af24d 
* Simple *loose* migration again... dropped the collection after removal of active.

Ref:
* `mongorestore --gzip --db openuserjs_devel --archive=./dev/devDBdirty.gz`

Post OpenUserJS#1772
@Martii Martii mentioned this pull request Oct 23, 2020
Merged
Martii added a commit that referenced this pull request Oct 23, 2020
@Martii
Sync devDBdirty (#1773)
79d3d6e 
* Simple *loose* migration again... dropped the collection after removal of active.

Ref:
* `mongorestore --gzip --db openuserjs_devel --archive=./dev/devDBdirty.gz`

Post #1772

Auto-merge
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
bug You've guessed it... this means a bug is reported. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. DB Pertains inclusively to the Database operations. security Usually relates to something critical. stability Important to operations.
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Martii