-
-
Notifications
You must be signed in to change notification settings - Fork 326
Open
Labels
Good first issueSeeting up first time contributors for success!Seeting up first time contributors for success!Type: Technical issueImprove something that users won't noticeImprove something that users won't notice
Description
Describe the issue
There is an old, orphaned page in app management that allows you to configure targets (/admin/#/targets). While this may work, it will overwrite the local app settings file on the server, which will then in turn get overwritten when CD runs or when CHT Conf is manually run. This is a worst practice.
Describe the improvement you'd like
We should remove this page
Describe alternatives you've considered
We could maybe try to encourage two way sync of app settings server -> client and also server <- client
Good first issue
This ticket has the Good first issue label! This means it's been especially curated by other CHT contributors to be easy to work on for first time contributors per the docs.
To succeed on this ticket, please:
- Find the template, link and any other content used in the admin app
- remove all templates and scr files, remove any related tests
- ensure the app still compiles
Metadata
Metadata
Assignees
Labels
Good first issueSeeting up first time contributors for success!Seeting up first time contributors for success!Type: Technical issueImprove something that users won't noticeImprove something that users won't notice
Type
Projects
Status
🔍 In Review