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

[stable21] Add primary key for ratelimit table #30970

Merged
merged 4 commits into from
Mar 21, 2022
Merged

[stable21] Add primary key for ratelimit table #30970

merged 4 commits into from
Mar 21, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Feb 2, 2022 •
edited
Loading

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #30235

@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 21.0.8 milestone Feb 2, 2022
@nickvergessen
Copy link
Member

nickvergessen commented Feb 2, 2022 •
edited
Loading

  • Needs a version bump once stable is unblocked

@MichaIng MichaIng added the 2. developing Work in progress label Feb 3, 2022
@nickvergessen
Add a primary key to ratelimit_entries table
aec4179 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Make the DB query simpler (as we just deleted all other entries)
599fdb7 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Bump autoloader
0dc5e98 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Bump version
8c2a18e 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 10, 2022
@nickvergessen
Copy link
Member

Rebased and fixed

@blizzz blizzz merged commit 723c85b into stable21 Mar 21, 2022
@blizzz blizzz deleted the backport/30235/stable21 branch March 21, 2022 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blizzz blizzz blizzz approved these changes

@miaulalala miaulalala miaulalala approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl

@CarlSchwan CarlSchwan Awaiting requested review from CarlSchwan

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

@GretaD GretaD Awaiting requested review from GretaD

@LukasReschke LukasReschke Awaiting requested review from LukasReschke

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

@PVince81 PVince81 Awaiting requested review from PVince81

Assignees
No one assigned
Labels
3. to review Waiting for reviews bug
Projects
None yet
Milestone
Nextcloud 21.0.10
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@nickvergessen @blizzz @miaulalala @MichaIng