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

feat(wifaqul ulama): support wifaqul ulama uk prayer times #164

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

hsravat-4590
Copy link

This Pull Request adds support for Wifaqul Ulama UK's calculation criteria

It notably includes the following key changes:

  • New United Kingdom Calculation Method
  • Support for Aqrab Yaum for High Latitude Fajr Calculations during periods of perpetual twilight

Add new HighLatitudeFajrRule for Aqrabul Ayyam during High Latitude Perpetual Twilight.
Adds a new Calculation Method used by Wifaqul Ulama UK
…lculation

Add specification of Fajr Angle for Aqrabul Youm Calculation
Dhuhr Adjustment should be 4 minutes.
@hsravat-4590 hsravat-4590 marked this pull request as draft August 24, 2024 15:43
Don't invoke Aqrabul Youm resolver unless in perpetual twilight
@hsravat-4590 hsravat-4590 marked this pull request as ready for review August 24, 2024 16:08
Optimise TimeComponents.utcDate function to reduce line count in PrayerTimes.ts
Fix failures with Linter on Unit Tests.
@z3bi
Copy link
Contributor

z3bi commented Aug 25, 2024

Jazakallah khair for this addition to the library. I would request if you could put together a json file of prayer times published by this group to be included in our testing repo (https://github.com/batoulapps/adhan-js/tree/master/Shared/Times) so we can see if there is a difference in the times being calculated by the library and the official published times.

Please take a look at https://github.com/batoulapps/adhan-js/blob/master/Shared/Times/London-MoonsightingCommittee.json as an example. In it I have a copy of the original times and show where we had to adjust the times to account for rounding. It should be sufficient to include one day of prayer times for each month so we have a representative sample for the year.

Also, it seems that the formatting of the README was changed significantly, this is making it difficult to see what was changed. If you could revert those changes and only have the new lines added that would be much better inshAllah.

@hsravat-4590 hsravat-4590 marked this pull request as draft August 25, 2024 11:36
@hsravat-4590 hsravat-4590 marked this pull request as ready for review August 27, 2024 10:52
@hsravat-4590
Copy link
Author

@z3bi I've updated this PR with requested changes.

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