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

Enhanced holiday dates in Ghana #80

Merged
merged 20 commits into from
Feb 8, 2024
Merged

Enhanced holiday dates in Ghana #80

merged 20 commits into from
Feb 8, 2024

Conversation

Ikpeba4ll
Copy link
Contributor

@Ikpeba4ll Ikpeba4ll commented Jan 18, 2024

This PR enhances the holiday dates for Ghana.

Holidays are not observed on weekends so any holiday which falls on a weekend is pushed to the next monday

NB: *** There are no fixed dates for the Eid-Ul-Fitr and Eid-Ul-Adha because they are movable feasts.
The dates for their observation are provided by the Office of the Chief Imam in the course of the year.

@Ikpeba4ll
Copy link
Contributor Author

@Nielsvanpach

Another round of review.

@Ikpeba4ll
Copy link
Contributor Author

Ikpeba4ll commented Jan 31, 2024

Kindly review

@Nielsvanpach

@Ikpeba4ll Ikpeba4ll changed the title Holidays in Ghana Added holidays in Ghana Jan 31, 2024
src/Countries/Ghana.php Outdated Show resolved Hide resolved
src/Countries/Ghana.php Outdated Show resolved Hide resolved
src/Countries/Ghana.php Outdated Show resolved Hide resolved
@vyivewonuMonet
Copy link

PR comments resolved.

@Nielsvanpach

src/Countries/Ghana.php Outdated Show resolved Hide resolved
src/Countries/Ghana.php Outdated Show resolved Hide resolved
@Ikpeba4ll Ikpeba4ll changed the title Added holidays in Ghana Improved holidays date in Ghana Feb 1, 2024
@Ikpeba4ll Ikpeba4ll changed the title Improved holidays date in Ghana Enhanced holiday dates in Ghana Feb 1, 2024
Copy link
Member

@Nielsvanpach Nielsvanpach left a comment

Choose a reason for hiding this comment

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

Thanks!

tests/Countries/GhanaTest.php Outdated Show resolved Hide resolved
tests/Countries/GhanaTest.php Outdated Show resolved Hide resolved
@vyivewonuMonet
Copy link

@Nielsvanpach

When will this be merged?

@Ikpeba4ll
Copy link
Contributor Author

@Nielsvanpach

This PR has been in approved state for so long.

Hope you haven't forgotten about it

@@ -5,7 +5,7 @@
},
{
"name": "Constitution Day",
"date": "2024-01-07"
"date": "2024-01-08"
Copy link
Member

Choose a reason for hiding this comment

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

these changes are correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the holiday is on 7th which is sunday so we observed it on 8th,

@@ -5,7 +5,7 @@
},
{
"name": "Constitution Day",
"date": "2024-01-07"
"date": "2024-01-08"
Copy link
Member

Choose a reason for hiding this comment

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

these changes are correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the holiday is on 7th which is sunday so we observed it on 8th,

@@ -5,7 +5,7 @@
},
{
"name": "Constitution Day",
"date": "2024-01-07"
"date": "2024-01-08"
Copy link
Member

Choose a reason for hiding this comment

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

these changes are correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the holiday is on 7th which is sunday so we observed it on 8th,

Copy link
Member

@Nielsvanpach Nielsvanpach left a comment

Choose a reason for hiding this comment

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

Thanks

@Nielsvanpach Nielsvanpach merged commit 0ab3409 into spatie:main Feb 8, 2024
13 of 14 checks passed
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