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

Object Details UUID #585

Closed
itdependsnetworks opened this issue Jun 17, 2021 · 8 comments · Fixed by #933
Closed

Object Details UUID #585

itdependsnetworks opened this issue Jun 17, 2021 · 8 comments · Fixed by #933
Labels
good first issue Great issues for newcomers! type: feature Introduction of substantial new functionality to the application
Milestone

Comments

@itdependsnetworks
Copy link
Contributor

User Story

As Austin the Network Automation engineer I am often working within Postman building out my automation
I want to be able to easily visualize the UUID's of objects in the UI
so that when I am building queries in Postman I can quickly copy the UUID to test API calls.
I will know this is done when every detailed object page has the UUID visible.

As P.D. the Plugin Developer I am often working within Postman building out my plugin
I want to be able to easily visualize the UUID's of objects in the UI
so that when I am building queries in Postman I can quickly copy the UUID to test API calls.
I will know this is done when every detailed object page has the UUID visible.

As Nelly the Network Engineer I am often working in GraphQL building out my test queries
I want to be able to easily visualize the UUID's of objects in the UI
so that when I am building my GraphQL queries to test GraphQL against various devices and objects using the query variables
I will know this is done when every detailed object page has the UUID visible.

Database Changes (Optional)

No

External Dependencies (Optional)

No

@glennmatthews glennmatthews added the type: feature Introduction of substantial new functionality to the application label Jun 17, 2021
@glennmatthews
Copy link
Contributor

Thanks for the feature request! In my opinion this would best be implemented as a part of #479.

@jedelman8
Copy link
Contributor

@glennmatthews did this by chance make it into #479 ?

@glennmatthews
Copy link
Contributor

To the best of my knowledge, it did not - it should be an easy addition to make though now that #479 is in place.

@glennmatthews glennmatthews added the good first issue Great issues for newcomers! label Aug 3, 2021
@jedelman8 jedelman8 added this to the v1.2.0 milestone Aug 6, 2021
@dgarros
Copy link
Contributor

dgarros commented Aug 30, 2021

I don't mind doing this one if we can agree on how we want it to look like
After looking at the code, I think it would be easy to add it to the created_updated.html template from core/teemplates/inc

We could add the UUID one the same line as created / updated info

Screen Shot 2021-08-30 at 3 58 14 PM

or we could add it below

Screen Shot 2021-08-30 at 3 57 13 PM

thoughts ?

@glennmatthews
Copy link
Contributor

It would belong in the base object_detail.html template rather than the inc/created_updated.html template, IMO. Do you think it's visible enough in that location/font? Just to throw another option out there, though perhaps this is too visible... :-)

image

@jedelman8
Copy link
Contributor

Do you think it's visible enough in that location/font?

How visible do we need this?

I do think those 3 ideas are too much in the user's face.

Here are a few more ideas:

What about the idea of creating a new table (always in the bottom left half of the template beneath tags) called "Details" or "Object Details" and for now, it has one row for UUID? This table could also include date created, last updated, etc. timestamps too.

Another idea is embedding this as the first line within the Change Log tab. Definitely wouldn't be front and center, but once users start to know it's there, they'd know right where it is

One final idea is putting it at the very bottom left not in a table, see below:

image

@jathanism
Copy link
Contributor

Do you think it's visible enough in that location/font?

How visible do we need this?

I do think those 3 ideas are too much in the user's face.

Here are a few more ideas:

What about the idea of creating a new table (always in the bottom left half of the template beneath tags) called "Details" or "Object Details" and for now, it has one row for UUID? This table could also include date created, last updated, etc. timestamps too.

IMO I think this is the best idea. It's a primary field (the primary key in fact), so it should be up top and prominent. And let's make sure it has a copy button like we have on Device.hostname in the Device detail view.

@glennmatthews
Copy link
Contributor

Implemented in next (upcoming 1.2 release) by #933.

glennmatthews added a commit that referenced this issue Sep 23, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Great issues for newcomers! type: feature Introduction of substantial new functionality to the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants