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

Fixes #1399 - Add Django Admin function to clear course updated dates. #1410

Merged
merged 4 commits into from
Sep 2, 2022

Conversation

jonespm
Copy link
Member

@jonespm jonespm commented Aug 26, 2022

I also added the date column to the list view.

I had a idea for adding it as a button but that required a custom template and more changes and this allows you to potentially select only specific courses so seems better. Less code and more functionality.

dashboard/admin.py Show resolved Hide resolved
dashboard/models.py Outdated Show resolved Hide resolved
Copy link
Member

@zqian zqian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removal of date_last_updated values works.

Approve the PR now, with the pending discussion of verbose_name.

Copy link
Contributor

@ssciolla ssciolla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't see any functional issues when I tested this, so I'll approve, you can merge it when you think the naming thing has been resolved (now or after some changes).

@jonespm jonespm merged commit 2435e43 into tl-its-umich-edu:master Sep 2, 2022
@jonespm jonespm deleted the issue_1399 branch September 2, 2022 19:48
jonespm added a commit to jonespm/student-dashboard-django that referenced this pull request Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add button to Django Admin to all admin to refresh all MyLA data
3 participants