bcgov / met-public

Modern Engagement Tool ops
Apache License 2.0
6 stars 19 forks source link

[To Main] Add try catch block around snowplow call #2521

Closed VineetBala-AOT closed 3 months ago

VineetBala-AOT commented 3 months ago

Issue #: https://apps.itsm.gov.bc.ca/jira/browse/DESENG-621

Description of changes: Added a try catch block to all snowplow calls

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

codecov-commenter commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 75.81%. Comparing base (8b2a453) to head (b51958e).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2521 +/- ## ========================================== - Coverage 75.83% 75.81% -0.03% ========================================== Files 597 597 Lines 21426 21431 +5 Branches 1575 1575 ========================================== - Hits 16249 16248 -1 - Misses 4920 4926 +6 Partials 257 257 ``` | [Flag](https://app.codecov.io/gh/bcgov/met-public/pull/2521/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov) | Coverage Δ | | |---|---|---| | [metweb](https://app.codecov.io/gh/bcgov/met-public/pull/2521/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov) | `63.89% <0.00%> (-0.04%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/bcgov/met-public/pull/2521?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov) | Coverage Δ | | |---|---|---| | [...-web/src/components/engagement/view/EmailModal.tsx](https://app.codecov.io/gh/bcgov/met-public/pull/2521?src=pr&el=tree&filepath=met-web%2Fsrc%2Fcomponents%2Fengagement%2Fview%2FEmailModal.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov#diff-bWV0LXdlYi9zcmMvY29tcG9uZW50cy9lbmdhZ2VtZW50L3ZpZXcvRW1haWxNb2RhbC50c3g=) | `52.94% <0.00%> (-2.17%)` | :arrow_down: | | [.../src/components/engagement/view/EngagementView.tsx](https://app.codecov.io/gh/bcgov/met-public/pull/2521?src=pr&el=tree&filepath=met-web%2Fsrc%2Fcomponents%2Fengagement%2Fview%2FEngagementView.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov#diff-bWV0LXdlYi9zcmMvY29tcG9uZW50cy9lbmdhZ2VtZW50L3ZpZXcvRW5nYWdlbWVudFZpZXcudHN4) | `78.18% <0.00%> (-2.96%)` | :arrow_down: | | [...gagement/view/widgets/Subscribe/EmailListModal.tsx](https://app.codecov.io/gh/bcgov/met-public/pull/2521?src=pr&el=tree&filepath=met-web%2Fsrc%2Fcomponents%2Fengagement%2Fview%2Fwidgets%2FSubscribe%2FEmailListModal.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov#diff-bWV0LXdlYi9zcmMvY29tcG9uZW50cy9lbmdhZ2VtZW50L3ZpZXcvd2lkZ2V0cy9TdWJzY3JpYmUvRW1haWxMaXN0TW9kYWwudHN4) | `41.17% <0.00%> (-2.58%)` | :arrow_down: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/bcgov/met-public/pull/2521/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgov)
sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud