Skip to content

Comments

feat: add unique constraint to email in t_roles#60

Closed
mvergez wants to merge 1 commit intoPnX-SI:developfrom
mvergez:feat/unique-email
Closed

feat: add unique constraint to email in t_roles#60
mvergez wants to merge 1 commit intoPnX-SI:developfrom
mvergez:feat/unique-email

Conversation

@mvergez
Copy link

@mvergez mvergez commented May 30, 2023

Dans le cadre d'une prestation pour l'ARB île de France, ajout d'une contrainte d'unicité sur l'adresse email suite à la création de plusieurs comptes avec la même adresse email

@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (93d3c1e) 38.59% compared to head (a8bcc3c) 38.59%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #60   +/-   ##
========================================
  Coverage    38.59%   38.59%           
========================================
  Files           12       12           
  Lines          710      710           
========================================
  Hits           274      274           
  Misses         436      436           
Flag Coverage Δ
pytest 38.59% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/pypnusershub/db/models.py 75.11% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mvergez
Copy link
Author

mvergez commented May 30, 2023

Je n'avais pas vu, cela à été déjà fait ici : #56
Désolé du doublon !

@mvergez mvergez closed this May 30, 2023
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.

1 participant