Skip to content

fix(MA): official names and announced Islamic dates - #690

Open
ilyassesalama wants to merge 2 commits into
commenthol:masterfrom
ilyassesalama:fix-ma-names-dates
Open

ilyassesalama wants to merge 2 commits into
commenthol:masterfrom
ilyassesalama:fix-ma-names-dates

Conversation

@ilyassesalama

Copy link
Copy Markdown

Builds on #689 (Amazigh New Year and Unity Day); until that is merged, this PR also shows its commit.

Names

  • French and Arabic names follow the official Moroccan wording (decree 2-04-426, the Ministry of Public Service list, maroc.ma, and the French and Arabic Wikipedia pages), e.g. "Aïd Al-Fitr" instead of "Fête de fin du Ramadan", "Aïd Al-Adha" instead of "Fête du mouton", "عيد الشغل", "فاتح محرم", "ذكرى المسيرة الخضراء".
  • English names of Morocco's own holidays follow the English Wikipedia page ("Throne Day", "Revolution Day", "Green March Day", …).
  • Shared holidays keep _name, so every other language keeps the shared translations; only the French or Arabic wording is overridden.

Dates
Islamic holidays in Morocco follow the moon sighting announced by the Ministry of Habous and Islamic Affairs, which often falls a day after the computed date. Fifteen past occurrences (2021–2026) are moved to the announced day with disable/enable, e.g. Eid al-Fitr 2025 on 31 March and Islamic New Year 2026 on 17 June. Future years stay computed.

Days off
One day per holiday, as decree 2-04-426 sets for the private sector (the civil service gets a second day for the Eids and Mawlid).

Sources
Replaced with official pages that load today: the decree on ILO NATLEX, the Ministry of Public Service list, maroc.ma (holiday list, Amazigh New Year, Royal Cabinet communiqué on Unity Day) and the Ministry of Habous.

MA fixtures regenerated with --writetests --countries MA; npm test passes.

This branch has not been deployed

No deployments
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.

1 participant