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

Actions/bindings/persistence docs track feature naming convention #297

Merged
merged 2 commits into from Feb 6, 2017

Conversation

watou
Copy link
Contributor

@watou watou commented Feb 6, 2017

To keep links to binding docs as they were and to track the feature name of bindings. The bindings/oh1 and bindings/oh2 subdirectories are now gone.

Fixes #295. This PR supercedes #296.

Should persistence and actions be moved up a level from their current “oh1” directory, since they don’t use the “1” appended to the feature name? If yes, what happens when new features for persistence or actions are added later in OH2?

Signed-off-by: John Cocula john@cocula.com

To keep links to binding docs as they were and to track the feature name of bindings.  The bindings/oh1 and bindings/oh2 subdirectories are now gone.

This PR supercedes openhab#296.

Should persistence and actions be moved up a level from their current “oh1” directory, since they don’t use the “1” appended to the feature name?  If yes, what happens when new features for persistence or actions are added later in OH2?

Signed-off-by: John Cocula <john@cocula.com>
@kaikreuzer
Copy link
Member

Should persistence and actions be moved up a level from their current “oh1” directory, since they don’t use the “1” appended to the feature name?

Yes, I would say so.

If yes, what happens when new features for persistence or actions are added later in OH2?

  • There won't be any 2.x actions, but rather rule modules for the new rule engine.
  • Persistence services should fully replace the 1.x versions as there is really merely a change in the namespace, but not from their functionality.

…ame will always mean the doc goes to `addons/${type}s/${id}`, this commit reflects this for actions, bindings and persistence service names.

An issue should be created to have Paper UI provide links to all add-ons, because it doesn’t appear to do so for non-OH2-native add-ons.

Signed-off-by: John Cocula <john@cocula.com>
@watou
Copy link
Contributor Author

watou commented Feb 6, 2017

The last commit should address #297 (comment).

@watou watou changed the title Binding docs track feature naming convention Actions/bindings/persistence docs track feature naming convention Feb 6, 2017
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!
Will rerun the update.sh as well...

@kaikreuzer kaikreuzer merged commit 20bee2c into openhab:gh-pages Feb 6, 2017
@watou
Copy link
Contributor Author

watou commented Feb 6, 2017

docs.openhab.org looks good from here.

@kaikreuzer
Copy link
Member

👍 Many thanks for your quick help!

@kaikreuzer
Copy link
Member

FTR: openhab/openhab-core#116

@watou watou deleted the track-feature-names branch February 20, 2017 10:01
@Confectrician Confectrician added this to the 2.x.x milestone Nov 18, 2022
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

Successfully merging this pull request may close these issues.

None yet

3 participants