Open dcarsey opened 3 years ago
i think i found why this fails.
the helper function checks converts the email to lower-case before comparison:
while the emails read from file are not:
this particular email is in the file with upper- and lower-case characters, so equality with the lower-cased version fails.
@dcarsey i submitted a pull request to address this. i didn't mean to take this issue from you, @rajshruti18 ! the code change was a one-liner, so i just went ahead and submitted it.
Rawr. This problems persists for Suzanne Aycock. Re-opening the tix.
From 7-14-21 == still didn't work
Suzanne.Aycock@duke.edu
https://heal-ctmd.renci.org/?status=success&username=suzanne.aycock@duke.edu%20&organization=Duke%20University&access_level=4&first_name=Suzanne&last_name=Aycock&email=suzanne.aycock@duke.edu%20
Check the README
REMEMBER THAT THESE ARE CASE SENSITIVE
Other info from our previous discussions on this:
I looked at the QI WRT how to add someone w/ HEAL write permissions. Since we are having trouble w/ adding this Suzanne person, and now that I’m reading the instructions and see “API” in it, I just want to ask out loud, as a layman…: if our API set-up has changed, does that mean that these instructions have changed???
To update list of heal users with write permissions: edit the heal_users.txt in the api folder and add the email address of the person in a newline who has write access now. Then run docker-compose -f docker-compose.prod.yml up --build -d -V --no-deps api so that the api reads the file again. (We may want a refresh endpoint at some point).