-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecating userList #161
Deprecating userList #161
Conversation
Production deployment of Parkour2 is ready to use this implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes are ok
typos + strip MPI information.
typos + strip MPI information.
I've already tried rebasing this 2 times, and I'm unable to merge due to conflicts... I will have to do a new PR since this was stale for too long... |
Ah, the issue was that |
This code needs to be tested (e.g.
args.project[0].split("_")[1]
, passed to the new function that replacesfetchFirstNameAndEmail
should be the 4-digit project ID).The new API endpoint is yet to be deployed to production. I'm only opening the PR as a draft on the functionality that we'll be able to use soon.