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/bulk/all metrics #2134

Merged
merged 5 commits into from
Dec 22, 2022
Merged

Feat/bulk/all metrics #2134

merged 5 commits into from
Dec 22, 2022

Conversation

gweiying
Copy link
Contributor

@gweiying gweiying commented Dec 13, 2022

Adds the following metrics for bulk QR code generation and bulk link creation:

  • bulk create count
  • bulk validation error (and error types tags)
  • custom query strings to track source for s3 bucket download

After release, to add S3 bucket logs and DD streaming for the 3 environments

@gweiying gweiying force-pushed the feat/bulk/all-metrics branch from 7de6d24 to d61dffd Compare December 13, 2022 03:06
@gweiying gweiying requested a review from halfwhole December 14, 2022 01:51
src/server/util/dogstatsd.ts Outdated Show resolved Hide resolved
src/server/util/dogstatsd.ts Outdated Show resolved Hide resolved
src/server/modules/bulk/BulkController.ts Outdated Show resolved Hide resolved
@gweiying gweiying requested a review from halfwhole December 21, 2022 02:44
@gweiying gweiying merged commit d256dd0 into develop Dec 22, 2022
@gweiying gweiying deleted the feat/bulk/all-metrics branch December 22, 2022 09:17
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