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

feat: date range support for sql reports #17

Merged
merged 14 commits into from
Jul 23, 2024

Conversation

tomwwinter
Copy link
Contributor

No description provided.

@tomwwinter tomwwinter force-pushed the tw/feat/2239-sql-reports-support branch from a7b41ed to f44221f Compare June 7, 2024 13:15
@tomwwinter tomwwinter force-pushed the tw/feat/2239-sql-reports-support branch from f44221f to d4d821e Compare June 7, 2024 13:25
@tomwwinter tomwwinter requested review from sleidig and removed request for sleidig June 14, 2024 08:56
@tomwwinter tomwwinter requested a review from sleidig June 14, 2024 09:07
Copy link
Member

@sleidig sleidig left a comment

Choose a reason for hiding this comment

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

Looks good to me, although I am not clear about the effects of a few of the changes.

Is there a deployed system for testing?

tomwwinter and others added 9 commits July 23, 2024 13:09
…/aambackendservice/reporting/domain/event/DocumentChangeEvent.kt

Co-authored-by: Sebastian <[email protected]>
…/aambackendservice/reporting/reportcalculation/controller/ReportCalculationController.kt

Co-authored-by: Sebastian <[email protected]>
…/aambackendservice/reporting/reportcalculation/controller/ReportCalculationController.kt

Co-authored-by: Sebastian <[email protected]>
…/aambackendservice/reporting/reportcalculation/controller/ReportCalculationController.kt

Co-authored-by: Sebastian <[email protected]>
# Conflicts:
#	application/aam-backend-service/src/main/resources/application.yaml
@tomwwinter tomwwinter merged commit c86abde into main Jul 23, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

support for sql reports without date range
2 participants