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

DASB-600 - Report sink fixes and updates #158

Merged
merged 11 commits into from
Aug 8, 2024
Merged

Conversation

mkrystof
Copy link
Collaborator

@mkrystof mkrystof commented Aug 8, 2024

No description provided.

Copy link

sonarcloud bot commented Aug 8, 2024

Copy link
Collaborator

@manu-govind manu-govind left a comment

Choose a reason for hiding this comment

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

Approved

@mkrystof mkrystof merged commit 5ab8502 into develop Aug 8, 2024
4 of 6 checks passed
@mkrystof mkrystof deleted the dev-report-sink-changes branch August 8, 2024 16:58
mkrystof added a commit that referenced this pull request Aug 20, 2024
* add the sh command

* add the sh command

* add the sh command

* gradle jib step added

* gradle jib step added

* ci build workflow files modified for graphql

* updated version

* Fixed name of dex_ingest_datetime (#144)

Co-authored-by: Matt B Krystof <[email protected]>

* Added submission projection for reports

* Added logic for submission projections

* Added submission projection for reports

* Added fileName to Provenance

* Added fileName to Provenance

* BOTFREY token changed

* Added SECURITY_ENABLED env var

* BOTFREY token changed

* Added function and property headers

* Added enum for StageInfo.Status and made TagsMap<String,String>

* Removed unused Tags class

* Removed unused Tags class
Modified Tags property to be Map<String,String>
Changed StageInfo.status to Enum

* Made changes based on PR review

* Made changes based on PR review

* Made changes based on Readme and the PR reviews

* Modified stage to action and updated the unit tests

* Made changes based on PR review

* Changed kotlin version to 1.9.24 (#151)

Co-authored-by: Matt B Krystof <[email protected]>

* Added dockerhub auth to gradle for jib

* Updated kotlin version in gradle properties and added dockerhub auth to jib

* DexIngestTimestamp to DexIngestDateTime

* Repalced dex_ingest_timestamp with dex_ingest_datetime

* new ci files

* new ci files

* github workflow files updating for 3 microservices

* github workflow files updating for 3 microservices

* github workflow files updating for 3 microservices

* github workflow files updating for 3 microservices

* Feature sort filter (#148)

* Filter and Sort

* Filter and Sort

* Set CosmosQueryRequestOptions

* Consolidate queries

* Use batch processing for query performance

* Use batch processing for query performance

* Use batch processing for query performance

* Update Query

* Update Query

* Update Query

* Update Query

* Sync with develop

* Remove submissionDate from the query.

* Renamed uploadDetails with getReports

* Added query to get raw reports (#156)

Co-authored-by: Matt B Krystof <[email protected]>

* Removed thanks.md file (#157)

Co-authored-by: Matt B Krystof <[email protected]>

* DASB-600 - Report sink fixes and updates (#158)

* Updated logging, reformatting

* First round of schema validation fixes

* Corrected upload-status schema

* Report schema validation updates

* Updated load testing Python script to latest version of report schemas

* Moving test files out of function app subfolder

* Fixed unit test broken by recent change

* Added missing dex_ingest_datetime field from reports

* Added dex ingest date/time to graphql service

* Fixed issues with Python test script for sending hl7 validation msg

* Fixed timestamps used in Python test script

---------

Co-authored-by: Matt B Krystof <[email protected]>

* Added new upload report schemas (#159)

Co-authored-by: Matt B Krystof <[email protected]>

* Fixed issue with the stage info start and end processing time format (#160)

Co-authored-by: Matt B Krystof <[email protected]>

* DASB-625 - Fixed issue with disposition type replace no longer working (#162)

* Updated Python load testing script to use disposition type

* Fixed issue with disposition type not searching report records correctly

---------

Co-authored-by: Matt B Krystof <[email protected]>

* Made stage info fields required and updated load testing script (#164)

Co-authored-by: Matt B Krystof <[email protected]>

* Dev feature upload status query updates (#163)

* Update Page Summary with the list of senderIds and jurisdictions

* Update the queries to use dexIngestDateTime.
Add the list of senderIds, Jurisdiction to the getUploadStatus api response.

* Update description

* Allow the content-types, "application/json" and "json"
Add new sort fields, "service" and action
Update detailed descriptions for Page Summary

* Reverted the changes to the default SchemaDefinition values in the files Metadata files.

Updated the default SchemaDefinition values for UploadStage

* Updated descriptions for UploadStatus

* Improved descriptions
Removed duplicate lines

* DASB-650 - Safe parse messages as reports, code cleanup (#165)

* Safe parse messages as reports, code cleanup

* Code cleanup

* Fixed load testing script fields for message metadata, tags and data

* Fixed the node names for safe message parsing of stage info, tags and data

* Fixed python load test script to use integer for message index rather than string

* Fixed stage info and message metadata serialization issues

---------

Co-authored-by: Matt B Krystof <[email protected]>

* Added missing report fields to queries (#166)

Co-authored-by: Matt B Krystof <[email protected]>

* Addresses the depreciation warning for datetime.utcnow() (#161)

* Addresses the depreciation warning for datetime.utcnow()

* updated output for dex_ingest_datetime

---------

Co-authored-by: Subbu Vemula <[email protected]>
Co-authored-by: Manu Kesava <[email protected]>
Co-authored-by: Matt B Krystof <[email protected]>
Co-authored-by: uek3-cdc <[email protected]>
Co-authored-by: Manu Govind Kesava Pillai <[email protected]>
Co-authored-by: Madhavi Tammineni <[email protected]>
Co-authored-by: Tea Smith <[email protected]>
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