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

emails-tweaks #82

Merged
merged 1 commit into from
Nov 12, 2024
Merged

emails-tweaks #82

merged 1 commit into from
Nov 12, 2024

Conversation

Jonathan-Adly
Copy link
Contributor

Description

Minor changes to the email logic when upsertion fails to match cloud.

Type of change

  • 🐛 Bug fix - fixes an issue without changing functionality (triggers PATCH version bump)

Release Impact

  • Current version: 1.3.1
  • Next version will be: 1.3.2

Checklist

  • My code follows the style guidelines
  • My code is tested with 99%+ coverage
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have updated the documentation to reflect my changes
  • I have added/updated tests to cover my changes

Breaking Changes

N/A

Migration Guide

N/A

@Jonathan-Adly Jonathan-Adly added bug Something isn't working patch PATCH (x.x.X): Bug fixes and minor changes labels Nov 12, 2024
@Jonathan-Adly Jonathan-Adly merged commit cca985d into main Nov 12, 2024
1 check passed
Copy link

codecov bot commented Nov 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
unittests 99.69% <100.00%> (+<0.01%) ⬆️

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

Files with missing lines Coverage Δ
web/api/tests/tests.py 100.00% <ø> (ø)
web/api/views.py 100.00% <100.00%> (ø)

@Jonathan-Adly Jonathan-Adly deleted the email-tweaks branch November 12, 2024 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working patch PATCH (x.x.X): Bug fixes and minor changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant