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

Provide optional parameters to Countries #168

Closed
wants to merge 3 commits into from

Conversation

inDeev
Copy link
Contributor

@inDeev inDeev commented Jan 23, 2024

This PR allow to provide any required parameters to countries models like region, ethnical type, ...

This PR was created by discussion about North Macedonia holidays Thread here so NorthMacedonia has some preparations made, which will be updated by its contributor

This PR also updating French model and test, as there was problem with custom France constructor just to use region (now it uses global one.

@Nielsvanpach
Copy link
Member

I don't think public holidays are based on someone's ethnic?

@Nielsvanpach
Copy link
Member

I don't really like this approach, so closing this one. Thanks for your work anyway!

@PetrKaterinak
Copy link

@Nielsvanpach Ethnic group is only one of possibilities of this PR, and if it is given by government as discussed in thread mentioned in original PR message, it is valid to have ethnical based holidays.

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