Skip to content

Add options to badges#10286

Merged
frenck merged 2 commits into
nextfrom
iantrich-patch-4
Sep 18, 2019
Merged

Add options to badges#10286
frenck merged 2 commits into
nextfrom
iantrich-patch-4

Conversation

@iantrich
Copy link
Copy Markdown
Member

@iantrich iantrich commented Sep 3, 2019

home-assistant/frontend#3552

Description:

Pull request in home-assistant (if applicable): home-assistant/home-assistant#

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next Home Assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

@probot-home-assistant probot-home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Sep 3, 2019
@iantrich iantrich requested a review from balloob September 3, 2019 15:50
balloob
balloob previously approved these changes Sep 3, 2019
Copy link
Copy Markdown
Member

@balloob balloob left a comment

Choose a reason for hiding this comment

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

ok to merge when the syntax error is fixed.

@frenck frenck added new-feature This PR adds documentation for a new Home Assistant feature to an existing integration parent-merged The parent PR has been merged already labels Sep 4, 2019
@iantrich iantrich self-assigned this Sep 10, 2019
@iantrich
Copy link
Copy Markdown
Member Author

@balloob I'm not seeing the error?

@balloob
Copy link
Copy Markdown
Member

balloob commented Sep 10, 2019

Neither do I, but netlify says

8:51:37 AM:   Liquid Exception: undefined method `split' for nil:NilClass in lovelace/views.markdown
8:51:37 AM: jekyll 3.8.6 | Error:  undefined method `split' for nil:NilClass
8:51:37 AM: Generating site failed

@iantrich
Copy link
Copy Markdown
Member Author

I have no idea what that means 😂

@iantrich
Copy link
Copy Markdown
Member Author

@frenck any ideas by chance?

Comment thread source/lovelace/views.markdown
@frenck frenck added this to the 0.99 milestone Sep 11, 2019
@frenck frenck changed the title Update views.markdown Add options to badges Sep 12, 2019
@frenck
Copy link
Copy Markdown
Member

frenck commented Sep 12, 2019

Hmm, I don't see a direct reason on why this fails. Will assign me and pick to one up later today. Interested to find out why this fails.

@frenck frenck self-assigned this Sep 12, 2019
@frenck
Copy link
Copy Markdown
Member

frenck commented Sep 18, 2019

@iantrich Found it! Although, this is more a workaround. Our configuration plugin assumes an integration/component/platform page, which in this case, it isn't. Providing an additional (meaningless) argument to the configuration tag fixes it for now (as it has been done before as it seems).

Need to revisit the part of the code. For now, this build should pass.

Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

🎉 Merging

@frenck frenck merged commit 7eea0a9 into next Sep 18, 2019
@probot-home-assistant probot-home-assistant Bot removed the parent-merged The parent PR has been merged already label Sep 18, 2019
@delete-merged-branch delete-merged-branch Bot deleted the iantrich-patch-4 branch September 18, 2019 11:16
@balloob balloob added the cherry-picked This PR has been manually picked and merged into the current branch label Sep 18, 2019
balloob pushed a commit that referenced this pull request Sep 18, 2019
* Update views.markdown

home-assistant/frontend#3552

* 🚑 Fixed build
@bbrendon bbrendon mentioned this pull request Sep 23, 2019
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked This PR has been manually picked and merged into the current branch has-parent This PR has a parent PR in another repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants