Next-prayer message: show time remaining, drop the timezone - #71
Merged
Conversation
/next now says how long remains until the prayer (e.g. "in 2 h 15 min"), localized in all 8 languages. The timezone suffix is dropped: it is the user's own configured timezone, so it carried no information. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What
/next(and the ⏭ menu button) now answers the real question — how long until the next prayer — instead of restating the user's own timezone.Before:
Fajr at 04:15 · Africa/CairoAfter:
Fajr at 04:15 · in 5 h 42 minHow
next_prayertemplate already had three placeholders in every locale, so the templates are untouched — the third argument is now a localized countdown instead ofprofile.Timezone.next_in_h/next_in_m/next_in_hmstrings for all 8 locales (en, ar, es, fr, ru, tr, uz, tt).Tests
untilNextunit test covering rounding-up, minutes-only, hours-only, and h+m rendering against the real English locale strings.next_prayersample args updated.make checkgreen (gofmt, vet, full test suite).🤖 Generated with Claude Code