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

Improve holiday management #462

Open
6 tasks done
naveensingh opened this issue Feb 1, 2025 · 3 comments
Open
6 tasks done

Improve holiday management #462

naveensingh opened this issue Feb 1, 2025 · 3 comments
Labels
enhancement New feature or request needs triage Issue is not yet ready for PR authors to take up

Comments

@naveensingh
Copy link
Member

naveensingh commented Feb 1, 2025

Checklist

  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the request will be dismissed otherwise.
  • This issue contains only one feature request.
  • I have read and understood the contribution guidelines.

Feature description

Current user flow for adding and removing holidays:

  • More options => Add holidays => Select country => Select holiday types => Import
  • More options => Settings => Manage event types => Delete "Holidays" event type

Proposed:

  • More options => Manage holidays => Select country => Select holiday types => Import.
  • More options => Manage holidays => Deselect country or deselect holiday types => Deselected holidays are automatically deleted.

Why do you want this feature?

Improved UI/UX.

Additional information

The UI could be similar to the following:

Image Image
@naveensingh naveensingh added enhancement New feature or request needs triage Issue is not yet ready for PR authors to take up labels Feb 1, 2025
@VidathD
Copy link

VidathD commented Feb 3, 2025

Are holidays currently unavailable for some countries? When I tried importing for Sri Lanka, no holidays were imported

@naveensingh
Copy link
Member Author

Yes, holidays for many countries are not yet added but Sri Lanka works for me.

@Aga-C
Copy link
Member

Aga-C commented Feb 3, 2025

@VidathD 4 countries are not supported by the holiday generator because of missing data in the upstream repo:

export const UNSUPPORTED_COUNTRIES = {
"IN": "India", // TODO: https://github.com/commenthol/date-holidays/issues/137
"KZ": "Kazakhstan", // TODO: (no GH issue)
"PK": "Pakistan", // TODO: https://github.com/commenthol/date-holidays/pull/138
"LK": "Sri Lanka", // TODO: (no GH issue)
};

Until it's supported, the best way is to manually add holidays for Sri Lanka by editing this ICS file: https://github.com/FossifyOrg/Calendar/blob/master/app/src/main/assets/holidays/LK/public.ics. We'll be happy if someone could contribute updated Sri Lankan holidays and also holidays for other unsupported countries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs triage Issue is not yet ready for PR authors to take up
Projects
None yet
Development

No branches or pull requests

3 participants