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 error when submitting short link with existing url suffix #664

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Fix error when submitting short link with existing url suffix #664

merged 2 commits into from
Nov 12, 2024

Conversation

Flo0807
Copy link
Collaborator

@Flo0807 Flo0807 commented Nov 12, 2024

Currently, there is an error when submitting a short link with a URL suffix that already exists.

  • Check for pkey unqiue constraint in changeset function
  • Drop unique index as this is already covered by the pkey index

@Flo0807 Flo0807 added the bug Something isn't working label Nov 12, 2024
@Flo0807 Flo0807 self-assigned this Nov 12, 2024
@pehbehbeh pehbehbeh merged commit c962207 into develop Nov 12, 2024
6 checks passed
@pehbehbeh pehbehbeh deleted the feature/properly-check-short-links-unique branch November 12, 2024 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pehbehbeh pehbehbeh Awaiting requested review from pehbehbeh

Assignees

@Flo0807 Flo0807

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Flo0807 @pehbehbeh