Skip to content
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

Add a date field to the courses table to capture when the course was added to MyLA #1135

Closed
jennlove-um opened this issue Dec 2, 2020 · 2 comments · Fixed by #1535
Closed
Assignees
Labels
django admin 📈 enhancement New feature or request

Comments

@jennlove-um
Copy link
Contributor

Describe your problem or feature you'd like added

Capture the date that an instructor adds MyLA to their course as an LTI. This data is useful for administrators in troubleshooting issues.

Describe the solution you'd like

Add a field to the courses table that captures the date and time that a course was added to MyLA.

@jennlove-um jennlove-um added this to To do in MyLA-Default-Project via automation Dec 2, 2020
@jennlove-um jennlove-um added this to To do in MyLA-2021.03.01 via automation Mar 11, 2021
@jennlove-um jennlove-um removed this from To do in MyLA-Default-Project Mar 11, 2021
@zqian zqian assigned zqian and unassigned ssciolla Sep 21, 2021
@jennlove-um jennlove-um removed this from To do in MyLA-2021.03.01 Sep 23, 2021
@jennlove-um jennlove-um added this to To do in MyLA-Default-Project via automation Sep 23, 2021
@jennlove-um jennlove-um added the 📈 enhancement New feature or request label Oct 12, 2022
@jennlove-um jennlove-um removed this from To do in MyLA-Default-Project Apr 27, 2023
@jennlove-um jennlove-um added this to To do in MyLA-2023.02.01 via automation Apr 27, 2023
@jonespm jonespm moved this from To do to In progress in MyLA-2023.02.01 Jun 15, 2023
jonespm added a commit to jonespm/student-dashboard-django that referenced this issue Sep 6, 2023
…o capture when the course was added to MyLA
jonespm added a commit to jonespm/student-dashboard-django that referenced this issue Sep 6, 2023
…o capture when the course was added to MyLA
jonespm added a commit to jonespm/student-dashboard-django that referenced this issue Sep 6, 2023
…o capture when the course was added to MyLA
jonespm added a commit to jonespm/student-dashboard-django that referenced this issue Sep 6, 2023
…o capture when the course was added to MyLA
MyLA-2023.02.01 automation moved this from In progress to Review/QA Sep 12, 2023
jonespm added a commit that referenced this issue Sep 12, 2023
@jonespm
Copy link
Member

jonespm commented Sep 12, 2023

PR merged Test Plan

  1. Verify in the admin interface that existing courses don't have any date created set
  2. Add a course, verify that it has the date that it was created
  3. Wait a few minutes, verify that the date hasn't changed

I feel like one edge case is that if this value is null and the course is saved it might add a date there. I think only an admin could do this and don't know how likely an issue is. It should be tested if this happens. I don't think an instructor changing the views should add a date but I didn't test this.

I think to avoid this we'd have to have it set the dates to something other than null by default. Like if they were 2018-01-01 for instance, before MyLA existed.

@jennlove-um
Copy link
Contributor Author

Tested in beta:

  • Tested adding a new course. The added date displayed correctly.
  • Tested an instructor changing the views on a course. No date was added to course with null date. No date was changed in course with date.
  • Tested admin updating course and saving in Django. No date was added to course with null date. No date was changed in course with date.

@jennlove-um jennlove-um moved this from Review/QA to Done in MyLA-2023.02.01 Sep 14, 2023
MyLA-2023.02.01 automation moved this from Done to Review/QA Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
django admin 📈 enhancement New feature or request
Projects
No open projects
MyLA-2023.02.01
  
Review/QA
4 participants