Skip to content

fix: correct jakim_code for 4 districts in Perak and Sarawak#5

Open
nurulizyansyaza wants to merge 1 commit into
mptwaktusolat:masterfrom
nurulizyansyaza:fix/zone-codes
Open

fix: correct jakim_code for 4 districts in Perak and Sarawak#5
nurulizyansyaza wants to merge 1 commit into
mptwaktusolat:masterfrom
nurulizyansyaza:fix/zone-codes

Conversation

@nurulizyansyaza

Copy link
Copy Markdown

Cross-referenced jakim_code values against official JAKIM zone-district mapping from the Takwim Solat reference document:
https://www.e-solat.gov.my/index.php?siteId=24&pageId=43 and JAKIM e-Solat API (https://www.e-solat.gov.my)

Fixes:

  • Kerian (Perak): PRK01 → PRK06
  • Kinta (Perak): PRK01 → PRK02
  • Hilir Perak (Perak): PRK02 → PRK05
  • Bukit Mabong (Sarawak): SWK03 → SWK04

Cross-referenced jakim_code values against official JAKIM zone-district
mapping from the Takwim Solat reference document:
https://www.e-solat.gov.my/index.php?siteId=24&pageId=43
and JAKIM e-Solat API (https://www.e-solat.gov.my)

Fixes:
- Kerian (Perak): PRK01 → PRK06
- Kinta (Perak): PRK01 → PRK02
- Hilir Perak (Perak): PRK02 → PRK05
- Bukit Mabong (Sarawak): SWK03 → SWK04
Copilot AI review requested due to automatic review settings March 4, 2026 06:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates JAKIM e-Solat zone codes in the district-level GeoJSON to match the official JAKIM zone↔district mapping (Perak + Sarawak), ensuring downstream consumers (e.g., prayer time lookups) use the correct zone identifiers.

Changes:

  • Corrected jakim_code for Bukit Mabong (Sarawak) from SWK03 to SWK04.
  • Corrected jakim_code for Hilir Perak, Kerian, and Kinta (Perak) to PRK05, PRK06, and PRK02 respectively.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@iqfareez

Copy link
Copy Markdown
Member

Thanks for the PR and apologies for didn't notice this earlier. Will review it by today.

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.

3 participants