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/container descriptions #382

Closed
wants to merge 3 commits into from
Closed

Conversation

vsoch
Copy link
Member

@vsoch vsoch commented Nov 11, 2021

This will add support for containers to have descriptions to close #381. We add the description with the container metadata (a json field) so no additional migrations to the database are necessary. Specifically, on the container details page if a user is an owner and has edit permission, they see an input with an update button:
Screenshot 2021-11-10 at 21-56-00 sregistry singularity registry

Otherwise they just see the description, verbatim:
Screenshot 2021-11-10 at 21-56-11 sregistry singularity registry

And the description (before truncation by the table) is added to the collections table:

Screenshot 2021-11-10 at 22-00-14 sregistry singularity registry

an owner will see an input in the container interface to add a description, and a non
owner will just see the description.

Signed-off-by: vsoch <[email protected]>
@xiaojinghu93
Copy link

Wow, @vsoch, thank you so much!
This new feature is awesome! Now we can move all the detailed info into the description area and that's greatly helpful.

Signed-off-by: vsoch <[email protected]>
@vsoch
Copy link
Member Author

vsoch commented Nov 15, 2021

@xiaojinghu93 I just updated to merge with main (no functionality changes just the CHANGELOG.md and VERSION file). Have you tested this locally and it is to your liking and ready to merge on successful tests again?

@vsoch
Copy link
Member Author

vsoch commented Oct 7, 2022

No response, closing.

@vsoch vsoch closed this Oct 7, 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 description for containers
2 participants