-
Notifications
You must be signed in to change notification settings - Fork 2
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
Kennel is not pulling correct user email addresses - notifications are not recieved #39
Comments
Changes happening in this branch: https://github.com/SquaredLabs/nucore-uconn/tree/add-kennel-seeder Once this commit's build succeeds, this line of code can be changed to read |
As of https://gitlab.com/squared-labs/kennel/-/jobs/129795827, Kennel now returns email addresses. Now to implement the import. |
So close. NoMethodError: undefined method `request_uri' for #<URI::Generic /count>
/var/www/nucore/lib/tasks/uconn.rake:14:in `do_kennel_call'
/var/www/nucore/lib/tasks/uconn.rake:23:in `block (2 levels) in <top (required)>'
/usr/share/rvm/gems/ruby-2.4.1/gems/rake-12.3.1/exe/rake:27:in `<top (required)>'
/usr/share/rvm/gems/ruby-2.4.1/bin/ruby_executable_hooks:15:in `eval'
/usr/share/rvm/gems/ruby-2.4.1/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => uconn:kennel_seed
(See full trace by running task with --trace) on https://github.com/SquaredLabs/nucore-uconn/blob/add-kennel-seeder/lib/tasks/uconn.rake. |
Forgot to configure the settings file in prod. Seems to be working now. |
@joshea0 Can you confirm that that the email functionality is working on prod.cider.core.uconn.edu? I never got a chance to test it with Rob on his prod branch. |
@CourtAW the seeder ran without error, but did produce a lot of warnings. I don't want to post the full log here since it has user info, but here is a sample line with the NetID replaced with a fake one:
My intuition is that these warnings are expected behavior because we don't have email for everyone in kennel, but I'm not sure how to verify if the email functionality is working correctly. |
@joshea0 once the kfs test is completed successfully, can we circle back to this issue? Would this be an issue/question we can bring to Table XI?
|
Circling back to this issue now. We have verified that the functionality for sending emails is working. The warning messages about people who have no email addresses is an issue in Kennel, so not a concern for TableXI. |
@robstolarz please add more detail as needed
The text was updated successfully, but these errors were encountered: