mattermost-webapp
mattermost-webapp copied to clipboard
mattermost
[MM-46537]: Yearly/Monthly toggle on Pricing model
Summary
Added a new monthly/yearly toggle to the pricing that changes the professional price accordingly. (Note: the prices shown in the pricing model are just from the mocked CWS so it's not the actual values right now)
Added telemetry for when the toggle is pressed as mentioned in MM-46565.
Ticket Link
Related Pull Requests
- Has server changes (please link here) : this is the new feature flag
- Has mobile changes (please link here)
Screenshots
| before | after |
|---|---|
 |
 |
https://user-images.githubusercontent.com/44761757/198726370-c88ab637-bda5-4a28-8922-5678db435805.mov
Release Note
NONE
E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.
Mattermost test server with CWS created! 🎉
Access here: https://mattermost-webapp-pr-11473.test.mattermost.cloud
Account Type Username Password Admin [email protected] Cws@10102020
We need to enable the feature flag in this test server to be able to see the changes - it probably needs https://github.com/mattermost/mattermost-server/pull/21551 merged
Mattermost test server with CWS created! 🎉 Access here: https://mattermost-webapp-pr-11473.test.mattermost.cloud Account Type Username Password Admin [email protected] Cws@10102020
We need to enable the feature flag in this test server to be able to see the changes - it probably needs mattermost/mattermost-server#21551 merged
Is it still alright that the default AnnualSubscription feature flag is set to false in that PR. Can that value be manually update in the test server?
@emmyni does changing the interval also translate to the purchase modal?
@AGMETEOR That's a good point, I'll add that to the other PR since that one handles the purchase modal
@emmyni it is still not showing up for me in the test server for some reason
generally looks great! The only thing I see that might be off is the spacing - The toggle should be 44px from the modal header and 44 px from the plan below
generally looks great! The only thing I see that might be off is the spacing - The toggle should be 44px from the modal header and 44 px from the plan below
@annelieseklein It should already be 44px
Closed and is now available here: https://github.com/mattermost/mattermost-webapp/pull/11534