Skip to content

Add Islamic Salah app - #3254

Open
egemenuzunali wants to merge 2 commits into
tidbyt:mainfrom
egemenuzunali:add-islamic-salah
Open

egemenuzunali wants to merge 2 commits into
tidbyt:mainfrom
egemenuzunali:add-islamic-salah

Conversation

@egemenuzunali

Copy link
Copy Markdown

Islamic Salah

Displays Fajr, Dhuhr, Asr, Maghrib and Isha for a location chosen in the Tidbyt mobile app, highlighting the next prayer.

Configuration

  • Location — the place timings are calculated for.
  • Calculation method — Muslim World League, ISNA, Umm al-Qura, Diyanet or Moonsighting Committee.
  • Asr calculation — Standard/Shafi or Hanafi.

Relationship to existing apps

There are two existing prayer-time apps, islamicprayer and prayer_times. This one is submitted under distinct id, name and packageName, and differs in offering a Hanafi/Shafi Asr selection, which neither existing app exposes — the two schools can place Asr around an hour apart, so the distinction is material for Hanafi users. Happy to fold this into an existing app instead if maintainers would prefer that.

Notes

  • Timings come from the AlAdhan API, cached for 15 minutes.
  • Coordinates are truncated to two decimal places before the request, so a precise device location is never sent to the provider.
  • pixlet check apps/islamicsalah/islamic_salah.star passes.

🤖 Generated with Claude Code

Displays Fajr, Dhuhr, Asr, Maghrib and Isha for a location chosen in the
mobile app, highlighting the next prayer. Configurable calculation method
and Standard/Shafi or Hanafi Asr. Coordinates are truncated to two decimal
places before being sent to the AlAdhan API.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@tidbyt

tidbyt Bot commented Aug 20, 2026 •

Copy link
Copy Markdown

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

Next Steps

Hello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:

  • Sign the CLA if you haven't already
  • Ensure your build is green! Any problem will display a proposed solution to try out
  • Get a review, either by Tidbyt Bot or by a Tidbyt engineer

Manual Review Required

Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:

Test Details
✅ App Dir All files are in a single app directory
🟡 Modules Usage of http.star requires review
✅ Original Author The original author matches the PR author

@tidbyt-bot

tidbyt-bot commented Aug 20, 2026 •

Copy link
Copy Markdown

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@egemenuzunali

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants