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.

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

refactor: remove the custom ErrDBUniqueConstraint error #197

Merged
merged 1 commit into from
Jun 17, 2023

Conversation

bfabio
Copy link
Member

@bfabio bfabio commented Jun 1, 2023

Remove common.ErrDBUniqueConstraint, which was used as a workaround for the lack of abstraction in gorm.

With the update in go-gorm/gorm#6004 we can now fully use gorm for db related errors (fe. gorm.ErrDuplicatedKey).

Remove common.ErrDBUniqueConstraint, which was used as a workaround
for the lack of abstraction in gorm.

With the update in go-gorm/gorm#6004 we can now
fully use gorm for db related errors (fe. gorm.ErrDuplicatedKey).
@bfabio bfabio force-pushed the gorm-err-unique-violation branch from d251666 to 0ad6a82 Compare June 17, 2023 22:10
@bfabio bfabio merged commit 60d2758 into main Jun 17, 2023
5 checks passed
@bfabio bfabio deleted the gorm-err-unique-violation branch June 17, 2023 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant