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

[refactor]: refactor handlers to be expandable #18

Closed
willianrod opened this issue Aug 20, 2023 · 2 comments
Closed

[refactor]: refactor handlers to be expandable #18

willianrod opened this issue Aug 20, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@willianrod
Copy link
Owner

Problem

The current architecture is expandable but requires a lot of configurations, we need it to be expandable with less configuration. This will allow ODeck to expand features very fast.

Possible solution

I still thinking about that, but I'm open to suggestions.

Minimum requirements

  • For every new handler, we should touch a single file
  • The handler should be localized using i18n
  • Registering the handlers should be done automatically
  • Every handler should add a new group to the sidebar
@willianrod willianrod added the enhancement New feature or request label Aug 20, 2023
@willianrod willianrod self-assigned this Aug 22, 2023
@willianrod
Copy link
Owner Author

Refactor is in progress complete. Got everything working in the new architecture. To develop a new handler you'll have to change one single file and add translations.

In some cases, some handlers will require some kind of configuration like defining an API key to make API requests. To allow this, I'll develop a settings page and only load the handler if the API key is defined.

@willianrod
Copy link
Owner Author

Refactor completed. Will be available on release v0.0.5 #32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

1 participant