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.

Sign up for GitHub

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

Jump to bottom

fix: dates for 2081 #60

Merged
merged 1 commit into from
Jun 3, 2024
Merged

fix: dates for 2081 #60

merged 1 commit into from
Jun 3, 2024

Conversation

RehmatFalcon
Copy link
Contributor

Some of the dates were wrong for 2081.

Month ❌ Wrong Month Days ✅ Correct Month Days
Jestha 31 32
Ashar 32 31
Falgun 30 29
Chaitra 30 31

I have added some tests for additional code as well.

mesaugat
mesaugat approved these changes Jun 3, 2024
View reviewed changes
Copy link
Member

@mesaugat mesaugat left a comment

Choose a reason for hiding this comment

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

Just skimmed through. Hopefully it is correct.

@mesaugat
Copy link
Member

mesaugat commented Jun 3, 2024

@RehmatFalcon Thanks for contributing!

@mesaugat mesaugat merged commit 1ae51bb into leapfrogtechnology:master Jun 3, 2024
@mesaugat mesaugat mentioned this pull request Jun 3, 2024
Closed
@mesaugat
Copy link
Member

mesaugat commented Jun 3, 2024

@RehmatFalcon Published version 2.0.2

@RehmatFalcon
Copy link
Contributor Author

@RehmatFalcon Published version 2.0.2

Thank you @mesaugat

@mesaugat mesaugat mentioned this pull request Jun 3, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mesaugat mesaugat mesaugat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@RehmatFalcon @mesaugat