-
Notifications
You must be signed in to change notification settings - Fork 80
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 a new time_pattern option. #242
Conversation
- Default new option value to 'HH:mm' - Remove time_format fallbacks to locale and instead fall through to using time_pattern. - time_format option can be removed when desired leaving time_pattern to act for time the same as date_pattern acts for date.
thanks for the PR and your contribution. i requested some changes to make sure we don't breaking existing installations of clock-weather-card after updating to the version including your changes |
I have all of these ready to add to a commit but have one question about the Full Configuration change before I update the pull request. Do you want it to be noted in any way in the Full Configuration section indicating time_pattern overrides time_format if both are provided? Something like: time_pattern: HH:mm Thanks for your feedback on this, this is my first time using TypeScript or contributing to HA, so it has been a slight learning curve. |
no problem, i'm glad your are contributing new features. |
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…up/plugin-node-resolve ^6.0.0
thanks for your contribution! |
Sorry, I hadn't commented again yet because I was going to see what I could do about that merge. I wanted a cleaner request without all the extra commits listed. Thanks for pulling this in. |
References #240