icgc-argo / program-service

GNU Affero General Public License v3.0
3 stars 0 forks source link

Add INTL as an allowed country code #345

Closed rosibaj closed 4 years ago

rosibaj commented 4 years ago

As you mentioned it, I think MUTO-INTL will better reflect the international nature of our project (cases coming from Africa, Asia, Europe, North and South America)

For projects from multiple countries we need to extend the allowed value on country code to also accept INTL

INTL needs to be added as a country code so -INTL is allowed

Location: https://github.com/icgc-argo/program-service/blob/develop/src/main/java/org/icgc/argo/program_service/validation/ProgramShortNameValidator.java

Expected Outcome

I can create a program with a shortname followed by -INTL as a country code. For testing, i will use ROSE-INTL.

image

blabadi commented 4 years ago

should we also validate that 1 country programs shouldn't end with INTL since it's now allowed ?

rosibaj commented 4 years ago

hmm i would say not a needed validation. Since only the DCC is making these then we can audit which ones we add this to

rosibaj commented 4 years ago

Tested on qa Sept 29,2020

image

@Buwujiu looks great!! thank so much.