Skip to content

Revert "send track events to crowd tenant"#1207

Merged
garrrikkotua merged 1 commit intomainfrom
revert-7
Aug 2, 2023
Merged

Revert "send track events to crowd tenant"#1207
garrrikkotua merged 1 commit intomainfrom
revert-7

Conversation

@garrrikkotua
Copy link
Copy Markdown
Contributor

@garrrikkotua garrrikkotua commented Aug 2, 2023

This reverts commit 18fe75e.

Changes proposed ✍️

What

🤖 Generated by Copilot at cb6bcd2

This pull request removes all the code related to the crowdAnalytics service, which is no longer used by the application. This includes deleting files, methods, types, and configuration variables that depend on the crowdAnalytics service.

🤖 Generated by Copilot at cb6bcd2

No more crowdAnalytics, we don't need it anymore
We purge the code of its traces, we make it clean and pure
We rebel against the service, we resist its obsolete allure
We free ourselves from its bondage, we break its chains for sure

Why

How

🤖 Generated by Copilot at cb6bcd2

  • Remove dependency on crowdAnalytics service and configuration (link, link, link, link, link, link, link)
  • Delete backend/src/segment/addProductDataToCrowdTenant.ts file that contained the logic for sending product analytics data to crowdAnalytics service (link)
  • Remove import and call to addProductData function in backend/src/segment/track.ts file that used the deleted file (link, link)
  • Remove crowdAnalytics configuration and type definition from backend/config/custom-environment-variables.json, backend/src/conf/configTypes.ts, and backend/src/conf/index.ts files that were no longer used by the application (link, link, link, link)
  • Remove unused getTenantInfo method from backend/src/database/repositories/tenantRepository.ts file that queried the tenant information from the database (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 97e0851 into main Aug 2, 2023
@garrrikkotua garrrikkotua deleted the revert-7 branch August 2, 2023 09:38
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