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
JobResult list view to show job Meta.name where available instead of class_path #775
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me! 🎉
I did leave a nitpicky comment, but I’m not sure it’s worth it or even an improvement, so feel free to disregard :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great except for the code duplication, which is a bit annoying; maybe we can just add a method to the class that does this resolution and call this at both sites?
You're absolutely right! I'll refactor it. |
Looks good to me now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* next: (37 commits) Add release-notes for nautobot#229 and nautobot#472 JobResult list view to show job Meta.name where available instead of class_path (nautobot#775) Bump version to 1.1.3-beta.1 Bump version and add release date Add release-notes for nautobot#785 and nautobot#786 Prioritize LoganImporter over built-in importers. See nautobot#785 (nautobot#788) Add release-note for nautobot#789 Remove url reverse lookup from nav menu initialization (nautobot#789) Add release-note for nautobot#758 Update job docs (nautobot#758) Custom Fields View (nautobot#735) Add release-notes for nautobot#742, nautobot#771, nautobot#773 Adding Logging Examples (nautobot#771) Process NAUTOBOT_DEBUG environment var (nautobot#742) Expanding the prometheus metrics docs (nautobot#773) Add release-note for nautobot#782 Update docs and examples to reflect invoke changes made by nautobot#584 (nautobot#782) Added release-notes for nautobot#723 and bump prerelease version Updates Powerfeed Utilization Data (nautobot#772) Bump version and release date. ...
Fixes: #472
In the JobResult list view, default to a column showing the associated Job's
Meta.name
string (where available) rather than theclass_path
. (Will fall back toclass_path
if the associated Job has been deleted or is otherwise unavailable to introspect). This makes it consistent with the JobResult detail view, which was already displaying theMeta.name
if available.I also removed the
job_id
UUID column from the table as being pretty much unnecessary.