Should external IDs be contrained to be unique?
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- Active
- Stack technique
- typescript
- Domaine
- database
Piste de recherche
Start by reading the participant import flows for the invite modal and CSV upload, then locate where the external ID is stored and validated. The issue raises competing concerns but does not settle whether IDs should be unique; first clarify the intended behavior with maintainers. Done would mean implementing the agreed behavior consistently across both import paths.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Currently there is no unique contraint on the external ID field when importing participants via invite modal or CSV upload.
Conceptually, that is strange to me as I'd expect an external ID to be an ID for a participant and therefore should be unique. Adding a unique constraint might help prevent accidental data input errors by study admins (or help catch bad data).
Conversely, we don't yet know how people are going to use the external id field, and so it might be okay to not have a unique constraint on that field.
I feel like the conservative position is safest until we get asked for a use-case for the alternative, but I'm happy to hear counter arguments!
- Langage dominant
- TypeScript
- Étoiles
- 4
- Forks
- 0
- Merge moyen
- 14 h 50 min
- PR mergées (30 j)
- 5
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Garvan-Data-Science-Platform/ctrl
-
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
-
admin-client backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
-
backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 commentaire ·
-
backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Toutes les issues de Garvan-Data-Science-Platform/ctrl
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
homarr-labs/homarr#7191 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
AvaProtocol/ava-sdk-js#276 ·
Les mainteneurs répondent en général sous 1 jour
-
useFragment does not require `variables` when the fragment's variables type has required fieldsOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
apollographql/apollo-client#13493 ·
Les mainteneurs répondent en général sous 1 jour