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

DOC-1658: reporting scheduled per scheme based on config #61

Open
wants to merge 70 commits into
base: main
Choose a base branch
from
Open

DOC-1658: reporting scheduled per scheme based on config #61

wants to merge 70 commits into from

Conversation

jbarnardeviden
Copy link
Contributor

No description provided.

@jbarnardeviden jbarnardeviden self-assigned this Dec 11, 2024
@jbarnardeviden
DOC-1658: handle scheduling for report
ff2c286
@jbarnardeviden
DOC-1658: create report csv and send with email via notify
6bc858b
@jbarnardeviden
DOC-1658: unit tests
41b4cdd
@jbarnardeviden
Merge branch main into DOC-1658-reporting
94f4992
@jbarnardeviden
Mocks for tests
afc9a94
@jbarnardeviden
Correct use of object
f8cfe54
@jbarnardeviden
Bump version
cde3d2b
@jbarnardeviden
Removed mocks from integration tests
676bd1d
@jbarnardeviden
Corrected changelog and sequelize operators usage
9d2ab0e
@jbarnardeviden
Tests for reporting index and additional filename test for publish by…
f7526b0 
… email
@jbarnardeviden
formatting
0294421
@jbarnardeviden
Corrected report database and added integration tests
3c21cf3
@jbarnardeviden
removed console log
ef41894
@jbarnardeviden
integration test for creating and sending report
efff5b9
@jbarnardeviden
Test report date ranges
5516fa1
@jbarnardeviden
Updated config to allow us to set the day of the month for montlhy ru…
3611392 
…ns and the day and month for yearly runs
@jbarnardeviden
Fix for running development
c712179
@jbarnardeviden
allow config of time of day to run report
e3fd395
@jbarnardeviden
spacing fix
61ee318
@jbarnardeviden
load test
37c7bb3
@jbarnardeviden
Config for report
808f136
@jbarnardeviden
WIP save file to blob storage
8e71a6d
@jbarnardeviden
Stream data from database, transform into CSV, and save in blob stroa…
72b4619 
…ge. Remove logic to send email.
@jbarnardeviden
WIP stream from db to blob storage
5afb20a
@jbarnardeviden
WIP CSV creation
2195a3c
@jbarnardeviden
WIP tidy up reporting changes
9f8cdcd
@jbarnardeviden
Removed test
05ecc8b
@jbarnardeviden
Updated tests
21dc817
@jbarnardeviden jbarnardeviden marked this pull request as ready for review January 20, 2025 16:24
GodsonLeigh
GodsonLeigh reviewed Jan 28, 2025
View reviewed changes
Copy link
Contributor

@GodsonLeigh GodsonLeigh left a comment

Choose a reason for hiding this comment

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

Just one comment about the constants

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
92.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

GodsonLeigh
GodsonLeigh approved these changes Jan 28, 2025
View reviewed changes
Copy link
Contributor

@GodsonLeigh GodsonLeigh left a comment

Choose a reason for hiding this comment

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

that-is-so-good-ella-payne

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GodsonLeigh GodsonLeigh GodsonLeigh approved these changes

Assignees

@jbarnardeviden jbarnardeviden

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jbarnardeviden @GodsonLeigh @ffcplatform