PnX-SI / UsersHub-authentification-module

Module Flask d'authentification de UsersHub
GNU General Public License v3.0
5 stars 13 forks source link

feat: add unique constraint to email in t_roles #60

Closed mvergez closed 1 year ago

mvergez commented 1 year ago

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[bot] commented 1 year ago

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 ```diff @@ 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](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://app.codecov.io/gh/PnX-SI/UsersHub-authentification-module/pull/60?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI) | Coverage Δ | | |---|---|---| | [src/pypnusershub/db/models.py](https://app.codecov.io/gh/PnX-SI/UsersHub-authentification-module/pull/60?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI#diff-c3JjL3B5cG51c2Vyc2h1Yi9kYi9tb2RlbHMucHk=) | `75.11% <100.00%> (ø)` | |

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

mvergez commented 1 year ago

Je n'avais pas vu, cela à été déjà fait ici : https://github.com/PnX-SI/UsersHub-authentification-module/pull/56 Désolé du doublon !