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

Add new externals, misc #516

Merged
merged 9 commits into from
Oct 18, 2017
Merged

Add new externals, misc #516

merged 9 commits into from
Oct 18, 2017

Conversation

ThomDietrich
Copy link
Member

@ThomDietrich ThomDietrich commented Sep 24, 2017

@kaikreuzer @mortommy @mboremski @ghys

Closes #62
Resolves #372
Resolves #481
Resolves #495
Resolves #505
Depends openhab/org.openhab.ui.habpanel#222

I missed this one: #480

This PR adds:

  • openHABian USAGE
  • alexa-skill USAGE
  • mycroft-skill USAGE
  • HABPanel config article
  • tidy-maven-plugin

Plus many small improvements

Thomas Dietrich added 8 commits September 24, 2017 16:32
Signed-off-by: Thomas Dietrich <thomas.dietrich@tu-ilmenau.de>
* openHABian USAGE
* alexa-skill USAGE
* mycroft-skill USAGE
* HABPanel config article

persistence -> persistences
io -> ios

Many minor changes to articles

Signed-off-by: Thomas Dietrich <thomas.dietrich@tu-ilmenau.de>
Signed-off-by: Thomas Dietrich <Thomas.Dietrich@tu-ilmenau.de>
Signed-off-by: Thomas Dietrich <Thomas.Dietrich@tu-ilmenau.de>
Signed-off-by: Thomas Dietrich <thomas.dietrich@tu-ilmenau.de>
Signed-off-by: Thomas Dietrich <Thomas.Dietrich@tu-ilmenau.de>
Signed-off-by: Thomas Dietrich <Thomas.Dietrich@tu-ilmenau.de>
Signed-off-by: Thomas Dietrich <Thomas.Dietrich@tu-ilmenau.de>
addons/uis.md Outdated
@@ -21,6 +23,13 @@ title: User Interfaces
| [iOS App]({{docu}}/addons/uis/apps/ios.html) | The native iOS app to access openHAB on the go. |
| [Windows 10 App]({{docu}}/addons/uis/apps/windows.html) | The native Windows 10 app to access openHAB on the go. |

## Iconsets

Basic UI, Classic UI and all apps present a graphical user interface as defined in [Sitemaps]({{base}}/configuration/sitemaps.html) by the help of [Items]({{base}}/configuration/items.html).
Copy link
Contributor

@bgilmer77 bgilmer77 Sep 25, 2017

Choose a reason for hiding this comment

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

Consider, "Basic UI, Classic UI and all openHAB apps present a graphical user interface as defined in Sitemaps , with the help of Items.

addons/uis.md Outdated
## Iconsets

Basic UI, Classic UI and all apps present a graphical user interface as defined in [Sitemaps]({{base}}/configuration/sitemaps.html) by the help of [Items]({{base}}/configuration/items.html).
Each Item respectively each Sitemap element may be assoziated with an icon visible next to it.
Copy link
Contributor

@bgilmer77 bgilmer77 Sep 25, 2017

Choose a reason for hiding this comment

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

I do not understand what you mean by this sentence - consider rewording it.
Consider changing "assoziated" to "associated".

addons/uis.md Outdated

Basic UI, Classic UI and all apps present a graphical user interface as defined in [Sitemaps]({{base}}/configuration/sitemaps.html) by the help of [Items]({{base}}/configuration/items.html).
Each Item respectively each Sitemap element may be assoziated with an icon visible next to it.
The Classic Icons Set is currently the only set of icons provided.
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider changing to, "Currently, only the Classic Icon Set is provided."

Signed-off-by: Thomas Dietrich <Thomas.Dietrich@tu-ilmenau.de>
@ThomDietrich
Copy link
Member Author

ThomDietrich commented Oct 9, 2017

Thanks @bgilmer77!

@kaikreuzer could you please have a final look at the changes to the head of pom.xml introduced through the tidy plugin (Motivation: the file had a few formatting errors when I first touched it) If you dislike the addition, we can remove it.
Afterwards please feel free to merge and issue a new content update. I'm on standby if anything comes up. Thanks!

(#480 will come with a later PR)

@ThomDietrich
Copy link
Member Author

@kaikreuzer just in case you missed the mentioning. If you don't have the time to do this prior to the SHD, the PR can just as well wait till next week.

@kaikreuzer
Copy link
Member

Hey @ThomDietrich, I am afraid you are right, I completely missed the mention here, sorry :-(

Just tested it and I see the meta-data on the headers being rendered on all add-ons pages:

screen shot 2017-10-18 at 16 01 40

Am I doing anything wrong?

@ThomDietrich
Copy link
Member Author

😲 weird.

Seems like the front matter part is rendered which I would guess happens because jekyll didn't identify it as that. Any obvious errors in the md file? I'll fire up my Vagrant machine...

@ThomDietrich
Copy link
Member Author

Just quickly went through the PR changes and don't think that any of them would have that effect.
Did you test this in your normal environment? I wonder if the issue is related to some weird encoding/lineendings quirk

@ThomDietrich
Copy link
Member Author

Tested, not seeing the issue.

  1. Checkout the PR
  2. Execute update-external-resources.sh
  3. Execute jekyll serve
  4. Success.

The only problem I encountered was that the iconset once again failed to build due to the missing data folder (commit not included in PR, no issue)

@kaikreuzer
Copy link
Member

Maybe an issue with my Jekyll version... Anyhow, the rest looks good, so let's assume that Github does it correctly.

@kaikreuzer kaikreuzer merged commit bd31110 into gh-pages Oct 18, 2017
@kaikreuzer kaikreuzer deleted the external-resources branch October 18, 2017 15:04
@kaikreuzer
Copy link
Member

Looks all good! 👍

@ThomDietrich
Copy link
Member Author

Yep, looks good. Thanks!

@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.

4 participants