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.

Sign up for GitHub

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

Jump to bottom

✨ [Frontend] Read product_ui_config from backend #7220

Merged
merged 21 commits into from
Feb 13, 2025
Merged

✨ [Frontend] Read product_ui_config from backend #7220

merged 21 commits into from
Feb 13, 2025

Conversation

odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Feb 12, 2025 •
edited
Loading

What do these changes do?

Make the frontend more flexible to what's now been moved to the db products.ui. It falls back to the still existing hardcoded ui_config.json (it will also be used to populated the DBs in different deployments)

UiConfig

Related issue/s

How to test

Dev-ops checklist

@odeimaiz odeimaiz self-assigned this Feb 12, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels Feb 12, 2025
@odeimaiz odeimaiz added this to the Singularity milestone Feb 12, 2025
@odeimaiz odeimaiz marked this pull request as ready for review February 13, 2025 10:45
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

pcrespov
pcrespov approved these changes Feb 13, 2025
View reviewed changes
Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

🎉

sanderegg
sanderegg approved these changes Feb 13, 2025
View reviewed changes
Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

👍

@odeimaiz odeimaiz merged commit 5f92eca into ITISFoundation:master Feb 13, 2025
57 checks passed
@odeimaiz odeimaiz mentioned this pull request Feb 14, 2025
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pcrespov pcrespov pcrespov approved these changes

@ignapas ignapas ignapas approved these changes

@GitHK GitHK GitHK approved these changes

@sanderegg sanderegg sanderegg approved these changes

@jsaq007 jsaq007 Awaiting requested review from jsaq007

@giancarloromeo giancarloromeo Awaiting requested review from giancarloromeo

@calys calys Awaiting requested review from calys

@mguidon mguidon Awaiting requested review from mguidon

@matusdrobuliak66 matusdrobuliak66 Awaiting requested review from matusdrobuliak66

Assignees

@odeimaiz odeimaiz

Labels
a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature
Projects
None yet
Milestone
Singularity
Development

Successfully merging this pull request may close these issues.
5 participants
@odeimaiz @ignapas @GitHK @pcrespov @sanderegg