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

feat: display packages versions in the side bar #225

Merged

Conversation

ntarocco
Copy link
Contributor

* display configured packages version at the bottom of the side bar
* closes inveniosoftware/invenio-app-rdm#2838
@ntarocco
Copy link
Contributor Author

Screenshot 2024-09-11 at 22 09 03

Copy link
Contributor

@jrcastro2 jrcastro2 left a comment

Choose a reason for hiding this comment

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

Reviewed with @sakshamarora1 LGTU! 🚀

@utnapischtim
Copy link
Contributor

Question: how are the versions of the packages added to the variable? is there a nice automatic solution or has it to be done by hand before of the deployment?

@utnapischtim
Copy link
Contributor

my two cents: a group name would be nice. we have "Site Management", "User Management" but the package -> version has no group name. maybe something like "Dev Management" and the sub name "Package versions" to make it open for other possible use cases.

@kpsherva
Copy link
Contributor

Question: how are the versions of the packages added to the variable? is there a nice automatic solution or has it to be done by hand before of the deployment?

@utnapischtim this is how it gets propagated: https://github.com/inveniosoftware/invenio-app-rdm/pull/2839/files#diff-988eb51a0bc0a8bf6efd0b04d1bb6b6d8f24cc80504ea36699a202904a20af4dR1428

https://github.com/inveniosoftware/cookiecutter-invenio-rdm/pull/294/files

@kpsherva kpsherva merged commit 136cbae into inveniosoftware:main Sep 26, 2024
4 checks passed
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.

Show packages versions in the administration panel
5 participants