Skip to content

Remove duplicate location in booking confirmation email#4682

Merged
miaulalala merged 1 commit intomainfrom
fix/double-location-issue
Oct 25, 2022
Merged

Remove duplicate location in booking confirmation email#4682
miaulalala merged 1 commit intomainfrom
fix/double-location-issue

Conversation

@miaulalala
Copy link
Contributor

Fixes #4681

@miaulalala miaulalala added 3. to review Waiting for reviews bug labels Oct 25, 2022
@miaulalala miaulalala self-assigned this Oct 25, 2022
@codecov
Copy link

codecov bot commented Oct 25, 2022

Codecov Report

Base: 41.84% // Head: 41.81% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (26f7de9) compared to base (9c9eac4).
Patch has no changes to coverable lines.

❗ Current head 26f7de9 differs from pull request most recent head 39cc2c8. Consider uploading reports for the commit 39cc2c8 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4682      +/-   ##
============================================
- Coverage     41.84%   41.81%   -0.03%     
+ Complexity      349      348       -1     
============================================
  Files           226      226              
  Lines          5662     5660       -2     
  Branches        738      738              
============================================
- Hits           2369     2367       -2     
  Misses         3293     3293              
Flag Coverage Δ
javascript 31.42% <ø> (ø)
php 69.05% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../calendar/lib/Service/Appointments/MailService.php 98.07% <0.00%> (-0.04%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Fixes #4681

Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala force-pushed the fix/double-location-issue branch from 26f7de9 to 39cc2c8 Compare October 25, 2022 08:22
@miaulalala miaulalala merged commit 3dbc53a into main Oct 25, 2022
@miaulalala miaulalala deleted the fix/double-location-issue branch October 25, 2022 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

Development

Successfully merging this pull request may close these issues.

How to edit the mail template for appointment?

3 participants