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

Keyless Entry Emails #1558

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Keyless Entry Emails #1558

wants to merge 30 commits into from

Conversation

ranta
Copy link
Collaborator

@ranta ranta commented Feb 13, 2025

🛠️ Changelog

  • Add Access Code info to Reservation Approved, Confirmed, Modified emails
  • Add new EmaiLTypes for Access Code Changed for Reservation and RecurringReservation
  • Refactor EmailType and contexts
  • Small misc fixes and changes

🧪 Test plan

  • Automated tests

🚧 Dependencies

  • None

🎫 Tickets

@ranta ranta added refactor Refactors something improvement Improves an existing feature labels Feb 13, 2025
@ranta ranta self-assigned this Feb 13, 2025
Copy link
Collaborator

@matti-lamppu matti-lamppu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes to suppressing errors from Pindora requests need to be added

@ranta ranta force-pushed the keyless-emails branch 2 times, most recently from 4ead8f7 to e0c6e14 Compare February 17, 2025 11:39
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
3.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing feature refactor Refactors something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants