| Summary: | Please add Lebanon to the list of holidays. | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] frameworks-kholidays | Reporter: | Hussam Al-Tayeb <ht990332> |
| Component: | general | Assignee: | kdepim bugs <kdepim-bugs> |
| Status: | REOPENED --- | ||
| Severity: | wishlist | CC: | kdepim-bugs, luigi.toscano |
| Priority: | NOR | ||
| Version: | 5.64.0 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Attachments: | draft holiday file | ||
|
Description
Hussam Al-Tayeb
2019-11-21 16:06:21 UTC
Created attachment 124065 [details]
draft holiday file
I tried to write a holiday file by imitating the eg_ar file but I'm not sure I did it correctly or how to install it.
Git commit 748d0219f3f34c729f21fe670030a3986c774342 by Allen Winter. Committed on 30/11/2019 at 14:50. Pushed by winterz into branch 'master'. add Lebanon holidays thanks for the contribution Hussam FIXED-IN: 5.65 M +1 -0 holidays/holidays.qrc A +52 -0 holidays/plan2/holiday_lb_ar https://commits.kde.org/kholidays/748d0219f3f34c729f21fe670030a3986c774342 Thank you very much, Allen. I will do my best to keep it maintained and up to date. Thanks Hussam - Allen, please commit using the correct authorship information. The file is not completely correct though, as a few mobile holidays (everything related to Easter, Western and Eastern) should be computed, not have a fixed day. Not sure about the Islamic ones, which use a different calendar (do we support it?) I'd suggest a revert and more time for review. The islamic ones are a different calendar but are official governmental holidays. One thing I can do is sync them monthly for each KF5 release. I'm not sure otherwise. Sorry. The question about the calendar is more for the KHolidays maintainers. Of course they are official Unfortunately updating is not a solution: the version of Frameworks may be used in different years, or not updated. You may define a way, but in general any recurrent day should be implemented with a formula. That said, other calendars are supported: see for example this file: https://cgit.kde.org/kholidays.git/tree/holidays/plan2/holiday_ar_es#n75 Can we please revert it? Right now half of the dates are not properly set. I suggest also the usage of phabricator to submit and discuss the review. I don't mind a revert. This was of course a draft :) Git commit df770235396d5e7c1dd98cdfeb02eea38660df14 by Luigi Toscano. Committed on 05/12/2019 at 14:18. Pushed by ltoscano into branch 'master'. Revert "add Lebanon holidays" As discussed in the bug, the change is not complete and having it as it is in the released Frameworks may be more problematic than not having it. Specifically, all the religious date should use the proper rule for dynamic calculation. This applies to both Christian holidays like Western and Eastern Easter and all Islamic holidays, which use a different calendar. The initial proposal is interesting but it should go through a few cycles of reviews, addressing the This reverts commit 748d0219f3f34c729f21fe670030a3986c774342. M +0 -1 holidays/holidays.qrc D +0 -52 holidays/plan2/holiday_lb_ar https://commits.kde.org/kholidays/df770235396d5e7c1dd98cdfeb02eea38660df14 You can find examples of islamic holidays in a couple files (holiday_bh_ar, holiday_dz_ar, holiday_eg_ar, holiday_ma_ar) (In reply to Christophe Giboudeaux from comment #9) > You can find examples of islamic holidays in a couple files (holiday_bh_ar, > holiday_dz_ar, holiday_eg_ar, holiday_ma_ar) This clarifies it a bit https://cgit.kde.org/kholidays.git/tree/holidays/plan2/holiday_bh_ar Thank you. |