Skip to content

Lovelace: correct glance config#5758

Merged
c727 merged 2 commits into
currentfrom
c727-ll-glance
Jul 15, 2018
Merged

Lovelace: correct glance config#5758
c727 merged 2 commits into
currentfrom
c727-ll-glance

Conversation

@c727
Copy link
Copy Markdown
Contributor

@c727 c727 commented Jul 11, 2018

Description:

correct glance config

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

@ghost ghost assigned c727 Jul 11, 2018
@ghost ghost added the ready-for-review This PR needs to be reviewed label Jul 11, 2018
@frenck frenck added Fix/Formatting/Language/Spelling current This PR goes into the current branch labels Jul 11, 2018
Comment thread source/_lovelace/glance.markdown Outdated
entities:
required: true
description: "A list of entity IDs or an `entity` object."
description: "A list of entity IDs or a `entity` objects."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

a entity objects doesn't make sense, maybe entity objects

fabaff
fabaff previously requested changes Jul 11, 2018
name:
required: true
description: "A name for `the entity_id`."
type: string
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

According to sample below those are valid keys. From my point of view those needs to be documented.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The keys there are optional. Yep, I will add a new setion for this

name:
required: false
description: Overwrites friendly name.
type: string
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In the entities card docs it says that type is an entity option, but it's not listed here. Is an entity different in different cards?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, it's different

Copy link
Copy Markdown
Contributor

@DubhAd DubhAd left a comment

Choose a reason for hiding this comment

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

👍

@c727 c727 merged commit a95ea42 into current Jul 15, 2018
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Jul 15, 2018
@c727 c727 deleted the c727-ll-glance branch July 15, 2018 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants