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

Fix : extended notification emiter #98

Merged
merged 1 commit into from
Jul 29, 2024
Merged

Fix : extended notification emiter #98

merged 1 commit into from
Jul 29, 2024

Conversation

soleil00
Copy link
Collaborator

No description provided.

Copy link

railway-app bot commented Jul 26, 2024

This PR was not deployed automatically as @soleil00 does not have access to the Railway project.

In order to get automatic PR deploys, please add @soleil00 to the project inside the project settings page.

@soleil00 soleil00 requested review from Heisjabo and yvanddniyo July 26, 2024 15:36
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 40.59406% with 60 lines in your changes missing coverage. Please review.

Project coverage is 72.46%. Comparing base (13f184d) to head (56ce458).

Files Patch % Lines
src/controllers/getOrderController.ts 11.11% 24 Missing ⚠️
src/controllers/paymentController.ts 9.52% 19 Missing ⚠️
src/services/payment.service.ts 54.54% 15 Missing ⚠️
src/controllers/wishesController.ts 92.85% 1 Missing ⚠️
src/events/handler.ts 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #98      +/-   ##
==========================================
- Coverage   72.53%   72.46%   -0.07%     
==========================================
  Files         130      130              
  Lines        2370     2408      +38     
  Branches      304      312       +8     
==========================================
+ Hits         1719     1745      +26     
- Misses        631      663      +32     
+ Partials       20        0      -20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@teerenzo teerenzo merged commit 7f189f8 into dev Jul 29, 2024
2 checks passed
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.

3 participants