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

Allow user to configure the integration with UI #94

Open
bcutter opened this issue Feb 6, 2024 · 5 comments
Open

Allow user to configure the integration with UI #94

bcutter opened this issue Feb 6, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@bcutter
Copy link

bcutter commented Feb 6, 2024

TBH this rather is a question in the first place. If the answer is "yes, still needed", this issue becomes a feature request.

Question: Is the usage of configuration.yaml following setup how-to https://github.com/dylandoamaral/trakt-integration?tab=readme-ov-file#2-update-configuration-file actually really/still needed?

Asking because "the other" trakt.tv integration is fully configurable from the UI:
https://github.com/custom-components/sensor.trakt?tab=readme-ov-file#installation-in-home-assistant

Add the Integration from the UI and setup your options

Please note this is some kind of a "deal breaker" question/thing. The other trakt.tv integration became kind of useless as it needs to be updated to make use of the current api (api.trakt.tv). Lots of users coming in, one referenced your integration here as a potential alternative and... well, here we are. Looking at oldschool .yaml hacking which is why I did not give it a shot yet. GUI is the future.

custom-components/sensor.trakt#72

@dylandoamaral
Copy link
Owner

Will have to find how to implement the UI part but it is something the project should have.

@dylandoamaral dylandoamaral changed the title Configuration.yaml still needed? allow user to configure the integration with UI Feb 9, 2024
@dylandoamaral dylandoamaral changed the title allow user to configure the integration with UI Allow user to configure the integration with UI Feb 9, 2024
@bcutter
Copy link
Author

bcutter commented Feb 10, 2024

Have a look here to start:

https://developers.home-assistant.io/docs/config_entries_config_flow_handler

Thanks for an estimation. As mentioned, many users have to make a decision now as the other integration is broken (but able to set up using the UI).

@dylandoamaral dylandoamaral added the enhancement New feature or request label Feb 13, 2024
@bcutter
Copy link
Author

bcutter commented Mar 10, 2024

@dylandoamaral do you have any rough estimation? Any help needed in testing etc.?

@dylandoamaral
Copy link
Owner

No I don't sorry, it is a huge work and for the moment I don't have time. It is a community project maybe someone will take the lead who knows.

@bcutter
Copy link
Author

bcutter commented Apr 2, 2024

No I don't sorry, it is a huge work and for the moment I don't have time. It is a community project maybe someone will take the lead who knows.

I hope so. The other (original / first) integration can be considered dead due to the missing adaptations to the trakt.tv API changes. Unfortunately there's no one from that project who could team up with you to make some consolidated progress here.

Anyway I still hope this basic GUI thing will be implemented. It blocks me and many other users from switching to this integration as in 2024 using YAML to configure an integration is really not the way to go anymore. Based on my experience an integration which can be set up from the UI has a much larger potential user base as it's so much more elegant and intuitive to use.

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
None yet
Development

No branches or pull requests

2 participants