Skip to content

Release 4.2.1#23917

Merged
sampaiodiego merged 7 commits intomasterfrom
release-4.2.1
Dec 10, 2021
Merged

Release 4.2.1#23917
sampaiodiego merged 7 commits intomasterfrom
release-4.2.1

Conversation

@sampaiodiego
Copy link
Member

2021-12-10 · 1 🎉 · 4 🐛 · 1 🔍 · 7 👩‍💻👨‍💻

Engine versions

  • Node: 12.22.1
  • NPM: 6.14.1
  • MongoDB: 3.6, 4.0, 4.2, 4.4, 5.0
  • Apps-Engine: 1.28.1

🎉 New features

🐛 Bug fixes

  • Error when creating an inactive user in admin panel (#23859)

    • Fix usersInRole array used to send email to activate a user.
  • Segmentation fault on CentOS 7 due to outdated sharp (#23796)

    Upgrades sharp to avoid a segmentation fault on CentOS 7 during startup related to sharp.node being loaded via process.dlopen().

    Suggested as a fix for versions 4.0.x and 4.1.x.

  • teams.removeMembers client usage (#23857)

  • Wrong button for non trial apps (#23861)

    This PR solves a bug on the marketplace that was happening with WhatsApp where it was displaying a trial button even though it didn't have a free trial period. The new verification I've added checks if the app is subscription-based and then checks if it has 0 trial days in all of its tiers. If it does, it shows a subscribe button. If it doesn't, it displays a trial button. Also, I've exposed the itsEnterpriseOnly flag as an extra measure in the case of apps like Facebook Messenger that are enterprise-only and consequently should show the subscribe button.
    Before:
    image
    After:
    image

🔍 Minor changes
  • Chore: Update Livechat (#23913)

👩‍💻👨‍💻 Core Team 🤓

@sampaiodiego sampaiodiego requested review from a team December 10, 2021 11:48
Copy link
Contributor

@casalsgh casalsgh left a comment

Choose a reason for hiding this comment

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

LGTM

@sampaiodiego sampaiodiego merged commit 98c75cf into master Dec 10, 2021
@sampaiodiego sampaiodiego deleted the release-4.2.1 branch December 10, 2021 14:44
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.

9 participants

Comments