You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Two issues were noticed in the models.py file that could be improved.
If the verbose_name matches the name of the field just remove the verbose name as it automatically matches. This follows Django's Less Code and DRY principles.
If the verbose name could be improved we should do that.
Additionally we don't use help_text to provide even more additional text on forms. It's possible some of these fields could have useful text. This should be figured out with @jennlove-um what these might be.
This appears under fields on the form like
I'm thinking we might want to have text like
Name, Start/End Date: "This field will be automatically populated by cron to match Canvas but can be adjusted if desired"
Data last updated: "This is the last time the cron was run and can be reset on the main courses page with the dropdown"
There's probably others., especially like the course view option area.
jonespm
changed the title
Clean up redundant and confusing names in models.
Clean up redundant and confusing names in models to improve admin interface
Sep 1, 2022
Two issues were noticed in the models.py file that could be improved.
If the verbose_name matches the name of the field just remove the verbose name as it automatically matches. This follows Django's Less Code and DRY principles.
If the verbose name could be improved we should do that.
Additionally we don't use help_text to provide even more additional text on forms. It's possible some of these fields could have useful text. This should be figured out with @jennlove-um what these might be.
This appears under fields on the form like
I'm thinking we might want to have text like
There's probably others., especially like the course view option area.
Originally posted by @jonespm in #1410 (comment)
The text was updated successfully, but these errors were encountered: