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

Add new script to process data streams for tenant #1144

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

garrrikkotua
Copy link
Contributor

@garrrikkotua garrrikkotua commented Jul 19, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 933cfc7

Added new methods and scripts to process integration data for tenants. The IntegrationDataRepo class can now query data by tenant id, and the package.json file has two new scripts to run the data processing tools for all or one tenant. These changes improve the integration data worker service functionality and testing.

🤖 Generated by Copilot at 933cfc7

Sing, O Muse, of the skillful work of the data engineers
Who devised clever scripts to process the integration data
From various sources they fetched and transformed the precious information
And stored it in the database, the treasure-house of IntegrationDataRepo

Why

How

🤖 Generated by Copilot at 933cfc7

  • Add two scripts to run the integration data worker service for testing and debugging (services/apps/integration_data_worker/package.json, link)
  • Add a method to fetch integration data ids for a given tenant (services/apps/integration_data_worker/src/repo/integrationData.repo.ts, link)
  • Add a private property to store the SQL query for the tenant data (services/apps/integration_data_worker/src/repo/integrationData.repo.ts, link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@garrrikkotua garrrikkotua merged commit 337f230 into main Jul 19, 2023
6 checks passed
@garrrikkotua garrrikkotua deleted the improvement/more-data-scripts branch July 19, 2023 12:02
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.

1 participant