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

✨ ABOUT REQUESTS #326

Closed
alexbelgium opened this issue May 10, 2022 · 8 comments
Closed

✨ ABOUT REQUESTS #326

alexbelgium opened this issue May 10, 2022 · 8 comments
Labels
prevent stale Prevents stale being applied

Comments

@alexbelgium
Copy link
Owner

Hi everyone,

When I was beginning to use HomeAssistant, I saw tons of possibilities with the system and was a bit frustrated when an app that I wanted was not available as an addon. I therefore started this repo. When people started asking me to port new apps that I was not using myself, I understood where they came from and tried to implement as many as possible.

Everyone is therefore free to make a request through the Issues system. However, I do not provide a guarantee neither of timings, nor of implementation. Globally, the key factors for implementation by order of priority are :

  • Is the image from linuxserver.io ? They have the best dockerhub images, that are very easily converted in addons
  • Number of people interested in the addon
  • Quality of existing dockerhub image

Maintaining all those addons take a lot of time, although I recognize that people help me a lot through bug reports & proposing PR to improve the repo. So please don't hesitate to help improve things where you can, it will be much appreciated and will benefit everyone!

Have a nice day everyone

@alexbelgium alexbelgium pinned this issue May 10, 2022
@alexbelgium alexbelgium added wontfix This will not be worked on prevent stale Prevents stale being applied labels May 10, 2022
@KairuByte
Copy link
Contributor

KairuByte commented May 10, 2022

As someone with a longish standing request that is slowly being worked on, I appreciate all you do. Even if the end result is something that doesn't happen, the effort alone is appreciated. I personally do not expect anything, as you are doing this in your free time. You owe us nothing, after all. ^_^

I do have one suggestion though. Maybe move requests over into discussions? That way debate and voting would be easier in general, and the issues section could remain for actual issues.

@alexbelgium
Copy link
Owner Author

Good idea. I see that the HA community addon did this. I'll look at it as an improvement! I just want to keep visibility on the requests so I don't forget them :) I've also added a script that checks if issues or requests are open, and notify those on top of the relevant addon readme, I'll have to see if the github api exposes discussions in the same way as issues. Thanks for the proposal!

@alexbelgium alexbelgium removed the wontfix This will not be worked on label May 13, 2022
@alexbelgium alexbelgium changed the title ABOUT REQUESTS ✨ ABOUT REQUESTS Jul 22, 2022
@alexbelgium
Copy link
Owner Author

alexbelgium commented Aug 24, 2022

I don't think I'll be able to add new addons for the moment

@richardwonka
Copy link

richardwonka commented Sep 20, 2024

A note - and a question - on the documentation.

I love that the available options are all listed here, even with examples.

There is one piece of information missing: Where do I put these options? Which file do I need to edit to add them?

e.g.: I get the error message in the admin dashboard about missing the default phone region.

The documentation tells me that I can set default_phone_region: de

But where do I set this option?

@alexbelgium
Copy link
Owner Author

Hi, thanks for the feedback - options can be configured from the addon options page within homeassistant. Watch-out that if they are not used, they might be hidden.

image image

@richardwonka
Copy link

richardwonka commented Sep 20, 2024

That’s the odd thing. The example was not in the UI. When I added the option in yaml, the ui refused to save the config.
( I verified syntax )

@alexbelgium
Copy link
Owner Author

Well I guess you are talking about Nextcloud, which is the only add-on using this option. On my system I can see the option, and it is in the config, so I'm sure it exists. Now the only reason that you wouldn't see it in the nextcloud add-on options were if that option is hidden and needs to be shown by the switch in bottom right of my first screen shot

@richardwonka
Copy link

richardwonka commented Sep 20, 2024

Yes, it is about nextcloud, sorry I hadn’t specified.

I had used that switch for the hidden options, that was the first place I looked.

I’m running the latest available nextcloud add-on on current Homeassistant and it was not showing and refusing to let me add the option.

EDIT:

  1. I wasn’t using my eyes that well, I have found the option
  2. It’s case sensitive and refusing to save when the country code is supplied in lower case. - removing case sensitivity might be an easy way to reduce surprises.

Facit: all sorted now, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prevent stale Prevents stale being applied
Projects
None yet
Development

No branches or pull requests

3 participants