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

Fix weeky report generation #597

Merged
merged 2 commits into from
Mar 27, 2025
Merged

Fix weeky report generation #597

merged 2 commits into from
Mar 27, 2025

Conversation

dmohns
Copy link
Member

@dmohns dmohns commented Mar 26, 2025

Brief summary of the change made

Couple of bug fixes to make

php artisan reports:city-revenue weekly

produce weekly reports and show them in the UI.

image

Are there any other side effects of this change that we should be aware of?

This does not yet fix generation of monthly report. These will be tackle in a separate step.

Describe how you tested your changes?

Pull Request checklist

Please confirm you have completed any of the necessary steps below.

  • Meaningful Pull Request title and description
  • Changes tested as described above
  • Added appropriate documentation for the change.
  • Created GitHub issues for any relevant followup/future enhancements if appropriate.

@dmohns dmohns requested a review from beesaferoot March 26, 2025 17:21
@dmohns dmohns merged commit 6972be8 into main Mar 27, 2025
12 checks passed
@dmohns dmohns deleted the fix-weekly-report-generation branch March 27, 2025 08:25
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.

2 participants