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

Enumerate current xhXXX app configs #372

Open
amcclain opened this issue Jul 19, 2024 · 0 comments
Open

Enumerate current xhXXX app configs #372

amcclain opened this issue Jul 19, 2024 · 0 comments

Comments

@amcclain
Copy link
Member

Keeping app config entries clean and up to date is difficult - we should make an extra effort to help keep our own Hoist-installed xhXXX app configs current.

I propose a careful review of our codebase across hoist-core and hoist-react to inventory every config we read so we can:

  • Ensure that all current configs are created with reasonable defaults in hoist-core Bootstrap
  • Provide an in-code list of these same configs

We could then leverage this list to alert admins to configs found in an app's DB that are no longer supported by the toolkit. Possible approaches there could be:

  • Query configs on app startup and log a warning for any unexpected DB configs
    • Could be exposed as a reusable method if apps wanted to impl the same pattern
  • More complex but potentially more actionable option would be to surface this info in hoist-react console - e.g. have that module get the list of supported configs and have some extra handling to flag them in the grid or toolbar.
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

No branches or pull requests

1 participant