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

Store correct date of bucket movement #244

Merged
merged 1 commit into from
May 25, 2024

Conversation

Lucaber
Copy link
Contributor

@Lucaber Lucaber commented May 20, 2024

fixes #240

@TheAxelander
Copy link
Owner

This would not work if you make a movement like for the previous month

@Lucaber
Copy link
Contributor Author

Lucaber commented May 20, 2024

Oh i didn't realize that this is possible

@TheAxelander
Copy link
Owner

I think the best way would be to add a month check. So if the current YearMonth is in the past, keep the old behavior (or use the last day of that month which would make more sense) otherwise use current date.

What do you think?

@Lucaber
Copy link
Contributor Author

Lucaber commented May 20, 2024

Yes, I`m now storing the last day of the month for previous or the first day for months in the future.

@TheAxelander
Copy link
Owner

Cool, I will have a look on it and do some testings in case of any further side effects.

@TheAxelander
Copy link
Owner

Testing looking good 👍

@TheAxelander TheAxelander merged commit 3c36c64 into TheAxelander:pre-release May 25, 2024
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.

Date of Bucket Movements always on the first of the month
2 participants