-
Notifications
You must be signed in to change notification settings - Fork 140
Ensure correct student names are in the application confirmation emails #955
Comments
Hi @ramonh I would love to work on this issue |
Hey @cesswairimu, sure thing, thank you so much for offering to help! Please make sure to let me know if I can help you in any way. |
@ramonh after rebasing I am getting this error. Tried to find some work around but nothing seems to work. |
Hey @cesswairimu Thanks for bringing this up! I tried checking out your branch and oddly enough, it seems to be working on my end. However, I'm on macOS so I suspect it could be a gem installation issue. Did I understand correctly that it worked before? Maybe one thing you could try is reinstalling the gems in your bundle. You can do that by running the following:
Hope this helps! |
yes, It worked before. I forked the repo a while back and after pulling the lastest changes, I am getting the error. Could be something to do with the upgrade to ruby-2.5.0 I am guessing. I just tried to fork the clone the repo directly and i am getting the same error. |
Very odd, and I take it re-installing the gems didn't work? |
Yep reinstalled successfully but the error persists |
Aw! Let's try a theory. Could you try opening the Then you can |
👍 That works Thanks a lot |
As quickly gone over in PR #951, we've had to temporarily disable application confirmation emails as we had an incident where students were sent the opposite applications with the opposite names.
We need to have a way to ensure that the correct
student_index
is used forstudent0
andstudent1
.The text was updated successfully, but these errors were encountered: