Skip to content

Conversation

@d-gubert
Copy link
Member

@d-gubert d-gubert commented Jan 4, 2022

Proposed changes (including videos or screenshots)

Introduction of new event IPreEmailSent was breaking the email function when the Apps-Engine framework was disabled in the administration

Issue(s)

Steps to test or reproduce

  • Disable the "Enable the App Framework" option in Admin > General
  • Send an email via Admin > Mailer

Further comments

@d-gubert d-gubert requested a review from sampaiodiego January 4, 2022 18:19
@d-gubert d-gubert added this to the 4.3.1 milestone Jan 4, 2022
@sampaiodiego sampaiodiego merged commit fc9c193 into develop Jan 4, 2022
@sampaiodiego sampaiodiego deleted the fix/apps-preemail-event branch January 4, 2022 21:26
@sampaiodiego sampaiodiego mentioned this pull request Jan 5, 2022
gabriellsh added a commit that referenced this pull request Jan 19, 2022
…hat into improve/setup-wizard

* 'improve/setup-wizard' of github.com:RocketChat/Rocket.Chat:
  Regression: Fix handling of http requests in apps bridge (#24211)
  Bump version to 4.3.2
  [FIX] Integration section crashing opening in My Account (#24068)
  Chore: Update Apps-Engine to 1.29.2 (#24171)
  [FIX] App Framework Enable hanging indefinitely (#24158)
  [FIX] CSV Importer failing to import users (#24090)
  Fix Engagement Dashboard API requests (#24142)
  [FIX] MAU when using micro services (#24204)
  [IMPROVE] Limit recent emojis to 27 (#24210)
  Bump version to 4.3.1
  Bump Fuselage packages
  [FIX][APPS] Action buttons not removed when app is disabled or uninstalled (#24107)
  [FIX][APPS] Prevents emails from being sent when apps framework is disabled (#24105)
  [FIX] Ensure Firefox 91 ESR support (#24096)
  Chore: Update Livechat (#24091)
  [FIX] Omnichannel enabled setting not working when creating rooms (#24067)
  [FIX] Enter not working on modal's multi-line input (#23981)
  [FIX] Omnichannel Current chats pagination not working (#24039)
@sampaiodiego sampaiodiego mentioned this pull request Jan 29, 2022
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