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

[openhabcloud] How to install the add-on #3162

Closed
wants to merge 1 commit into from

Conversation

dougmcdorman
Copy link

It wasn't obvious where to find this in the pile of add-ons so adding a one liner to tell people (beginners) where to find it.

It wasn't obvious where to find this in the pile of add-ons so adding a one liner to tell people (beginners) where to find it.
@martinvw martinvw requested a review from ThomDietrich January 23, 2018 17:50
@kaikreuzer
Copy link
Member

I would want to avoid adding such a line in every single add-on documentation - especially as Paper UI is just one mechanism, while there are other options as well.
It makes much more sense to have it explained ONCE in the docs (like the top of https://docs.openhab.org/addons/index.html) for all add-ons.

I assume your problems to find it were due to the "Misc" category. Possibly the docs menu should be better aligned with the Paper UI categories - it is not obvious that https://docs.openhab.org/addons/io.html corresponds to "Misc" and not all entries there actually are add-ons. @ThomDietrich, wdyt?

@ThomDietrich
Copy link
Member

ThomDietrich commented Jan 23, 2018

Hello @dougmcdorman 👋
thank you very much for your contribution.
This kind of addition to the README is imho not ideal. A user of openHAB should know how addons are managed and installed in openHAB. With that presumption in mind it is (a) counterintuitive and confusing to specifically go into those details in the add-on README and/or (b) make sure every/none other add-on READMEs contain the same details.

Long story short: I understand why you came here and are willing to add these details. How about a different approach. Could you rewrite your sentence to something along the line of

The "openHAB Cloud Connector" is an optional add-on and can be installed e.g. through the "Misc" add-ons section in Paper UI.

@kaikreuzer I was already typing when you posted your answer. As you can see I basically agree with you but also want to satisfy the problem @dougmcdorman was facing before going through the effort of creating the PR.

Regarding your question: Yes that is a fair point. There are quite a few other thing I currently dislike in the menu structure and I already thought of some improvement which I wanted to tackle as soon as eclipse-archived/smarthome#4503 was settled. This issue would go a long way in making a few things possible/easier in docs.openhab.org and in Paper UI (I suppose). It's a shame it isn't attracting as much attention as I expected...

Creating a docs issue for what you said right now.

@kaikreuzer
Copy link
Member

@ThomDietrich I understood your "Zaunpfahl" 😉

@martinvw
Copy link
Member

@dougmcdorman is this problem maybe already addressed by the improved documentation (structure)? If so are okay with closing this PR?

@wborn wborn changed the title How to install the openHAB Cloud add-on [openhabcloud] How to install the add-on Dec 18, 2018
@wborn
Copy link
Member

wborn commented Jan 27, 2019

Shall we close this PR and create issues in the docs/website project if it still remains unclear for people how to install each and every addon?

It should be possible for the website to generate some information for each addon how it can be installed in various UIs or using text files. I.e. similar to how it already generates a link for creating a PR for editing documentation.

@Confectrician
Copy link
Contributor

Confectrician commented Jan 30, 2019

+1 @wborn
this is really nothing that should be handled for each binding individually. :)

@wborn
Copy link
Member

wborn commented Jan 30, 2019

I've created openhab/website#152 for generating add-on installation instructions on the website.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants