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

ETQ expert, je dois confirmer mon mail #10705

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

lisa-durand
Copy link
Contributor

closes #10560

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

Attention: Patch coverage is 51.21951% with 20 lines in your changes missing coverage. Please review.

Project coverage is 84.58%. Comparing base (bf1a057) to head (fae81d0).
Report is 278 commits behind head on main.

Files Patch % Lines
app/models/user.rb 0.00% 12 Missing ⚠️
app/mailers/user_mailer.rb 14.28% 6 Missing ⚠️
app/controllers/instructeurs/avis_controller.rb 75.00% 1 Missing ⚠️
app/mailers/avis_mailer.rb 90.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10705      +/-   ##
==========================================
+ Coverage   80.23%   84.58%   +4.35%     
==========================================
  Files        1233     1118     -115     
  Lines       26263    24769    -1494     
  Branches     4716     4608     -108     
==========================================
- Hits        21071    20950     -121     
+ Misses       5192     3819    -1373     

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

@lisa-durand lisa-durand force-pushed the email-expert-need-confirmation-ldu branch from 82e1031 to 1bfa92e Compare August 22, 2024 09:55
@lisa-durand lisa-durand force-pushed the email-expert-need-confirmation-ldu branch from 1bfa92e to fae81d0 Compare August 22, 2024 10:09
@lisa-durand lisa-durand marked this pull request as ready for review August 22, 2024 12:53
@lisa-durand lisa-durand added this pull request to the merge queue Aug 22, 2024
Merged via the queue into main with commit 8e7e8d9 Aug 22, 2024
18 checks passed
@lisa-durand lisa-durand deleted the email-expert-need-confirmation-ldu branch August 22, 2024 13:01
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.

ETQ expert, je dois confirmer mon mail
2 participants