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 auto format feature #139

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

EdLeckert
Copy link
Contributor

Inspired by Issue #138, this PR allows users to take advantage of the default formatting provided by Home Assistant. This formatting observes the setting of display precision (currently only for entities, I believe) and unit of measurement for both entities and attributes, if defined.

Thinking of the short term and not wanting to create a breaking change, the feature currently defaults to off. It's likely that most users will already have their own manual formatting in place, and this feature would cause additional unwanted format changes. After upgrading, users can make adjustments to use the feature as time permits.

Once the feature is turned on globally for a table, it can be overridden for individual columns.

Documentation has been updated.

Any comments and assistance with testing would be appreciated.

@daringer
Copy link
Collaborator

uhhh, that's a pretty good idea, nice! will test today ...

@daringer daringer merged commit dc02a59 into custom-cards:master Jun 3, 2024
1 check passed
@daringer
Copy link
Collaborator

daringer commented Jun 3, 2024

the two recent PRs might already be worth a release, what do you think ?

@EdLeckert
Copy link
Contributor Author

I think so. We haven't had one since March, and the ability to disable sorting was a requested feature.

@daringer
Copy link
Collaborator

daringer commented Jun 3, 2024

released v0.7.7 - no obvious issues seen so far

@EdLeckert
Copy link
Contributor Author

Looks good to me.

@EdLeckert EdLeckert deleted the feat-format-using-hass branch July 8, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants