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

fix(systemtags): objecttype index registration #50200

Merged
merged 1 commit into from
Jan 15, 2025
Merged

fix(systemtags): objecttype index registration #50200

merged 1 commit into from
Jan 15, 2025
+10 −13

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv added this to the Nextcloud 31 milestone Jan 15, 2025
@skjnldsv skjnldsv self-assigned this Jan 15, 2025
@skjnldsv skjnldsv requested review from ArtificialOwl and artonge and removed request for a team January 15, 2025 15:58
@skjnldsv skjnldsv mentioned this pull request Jan 15, 2025
Closed
3 tasks
ChristophWurst
ChristophWurst approved these changes Jan 15, 2025
View reviewed changes
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Smooth

@skjnldsv skjnldsv enabled auto-merge January 15, 2025 16:01
@skjnldsv skjnldsv force-pushed the fix/index-systemtags branch from ad6f4a7 to 57ffca3 Compare January 15, 2025 16:03
provokateurin
provokateurin approved these changes Jan 15, 2025
View reviewed changes
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

Editing existing migrations 🙈

@skjnldsv
Copy link
Member Author

Editing existing migrations 🙈

This is the way

@skjnldsv skjnldsv disabled auto-merge January 15, 2025 16:32
@skjnldsv skjnldsv merged commit 0cad072 into master Jan 15, 2025
185 of 188 checks passed
@skjnldsv skjnldsv deleted the fix/index-systemtags branch January 15, 2025 16:32
@nickvergessen
Copy link
Member

Editing existing migrations 🙈

This is the way

Just for the record, only legit for adding indexes and optional columns to avoid "warning" about missing things right after installation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@provokateurin provokateurin provokateurin approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl was automatically assigned from nextcloud/server-backend

@artonge artonge Awaiting requested review from artonge artonge was automatically assigned from nextcloud/server-backend

Assignees

@skjnldsv skjnldsv

Labels
3. to review Waiting for reviews bug feature: tags technical debt
Projects
📁 Files team
Status: 🏗️ In progress
Milestone
Nextcloud 31
Development

Successfully merging this pull request may close these issues.
4 participants
@skjnldsv @nickvergessen @ChristophWurst @provokateurin