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

chore: deprecate admin UI menu items #7155

Merged
merged 4 commits into from
Jun 19, 2024
Merged

Conversation

naftis
Copy link
Collaborator

@naftis naftis commented Jun 14, 2024

#6975

Deprecated items in Admin UI that can now be configured directly from the country configuration. This helps us maintain the configuration options. Due to our experience in country installations, we've noticed that the functional configuration needs technical knowledge, which is required anyway when configuring forms in country configuration.

Note that this pull request doesn't remove the stale code left behind of these views. Some parts of the application like advanced search depend on some of the admin UI code, so it's being done in a separate PR #7156

@naftis naftis added this to the v1.5.0 milestone Jun 19, 2024
@naftis naftis merged commit a33e0b1 into develop Jun 19, 2024
21 checks passed
@naftis naftis deleted the deprecate-adminui-items branch June 19, 2024 06:26
github-actions bot pushed a commit that referenced this pull request Jun 19, 2024
naftis added a commit that referenced this pull request Jun 19, 2024
* chore: deprecate admin ui menu items

* chore(changelog)

* fix: remove tests for application and certificate

* chore(changelog): update wording
naftis added a commit that referenced this pull request Jun 19, 2024
* Placeholder commit for PR #7155

* chore: deprecate admin UI menu items (#7155)

* chore: deprecate admin ui menu items

* chore(changelog)

* fix: remove tests for application and certificate

* chore(changelog): update wording

---------

Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: Pyry Rouvila <pyry@opencrvs.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants