Skip to content

feat(db): Ensure that index names are unique across the database#39506

Merged
nickvergessen merged 2 commits intomasterfrom
techdebt/39488/ensure-index-name-uniqueness
Jul 24, 2023
Merged

feat(db): Ensure that index names are unique across the database#39506
nickvergessen merged 2 commits intomasterfrom
techdebt/39488/ensure-index-name-uniqueness

Conversation

@nickvergessen
Copy link
Copy Markdown
Member

Error when enabling Talk after Text was enabled (before nextcloud/text#4553 )

Index name "ts_session" for table "oc_talk_sessions" collides with the constraint on table "oc_text_steps".

Checklist

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 labels Jul 20, 2023
@nickvergessen nickvergessen added this to the Nextcloud 28 milestone Jul 20, 2023
@nickvergessen nickvergessen self-assigned this Jul 20, 2023
Copy link
Copy Markdown
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of that 💙

@nickvergessen nickvergessen enabled auto-merge July 20, 2023 13:35
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen disabled auto-merge July 24, 2023 12:44
@nickvergessen nickvergessen merged commit 7548e62 into master Jul 24, 2023
@nickvergessen nickvergessen deleted the techdebt/39488/ensure-index-name-uniqueness branch July 24, 2023 12:44
joshtrichards added a commit to nextcloud/documentation that referenced this pull request Feb 17, 2024
backportbot bot pushed a commit to nextcloud/documentation that referenced this pull request Mar 7, 2024
acataluddi pushed a commit to acataluddi/nextcloud-documentation that referenced this pull request Apr 28, 2024
Follow-on to nextcloud/server#39506 & nextcloud/server#43357

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Adriano Cataluddi <acataluddi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a check to the Schema to ensure indexes are unique

3 participants