feat: allow custom time format #214
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Nice thanks for the PR! 😄 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Allow the time format for the notification history & the notifications themselves to be customized.
Handy for those who don't do 24h clocks.
I did consolidate 2 formats:
https://github.com/rcarriga/nvim-notify/pull/214/files#diff-cc0264adaefdad4a3e145a4ad44a0c98f186d24da90e022e9215f3ef68fa828bL32
and
https://github.com/rcarriga/nvim-notify/pull/214/files#diff-3eb94df4dcd115e2877155af0a02b9cee18d96cc0b1ff3a908459b63fd6948cdL46
Since
%Tis%H:%M:%SI imagined that the fact that they differ wasn't intentional. Let me know if you want me to separate them.