mattermost-webapp icon indicating copy to clipboard operation
mattermost-webapp copied to clipboard

Remove integrations limits and usage

Open javaguirre opened this issue 1 year ago • 7 comments

Summary

This PR removes all integrations limits and usage.

Ticket Link

Related Pull Requests

Screenshots

Release Note

Removes all integration limits and usage limit components in the billing UI.

javaguirre avatar Oct 13 '22 08:10 javaguirre

I couldn't test how it's seen in the billing section because I don't have an enterprise license (the trial doesn't work on my machine either) to see it, but I'll try to get it to test this.

javaguirre avatar Oct 13 '22 09:10 javaguirre

Solved the license problem!

javaguirre avatar Oct 13 '22 10:10 javaguirre

Ditto that there's some more removal needed. I know this feature kind of sprawls throughout the webapp, so I made a PR with everything additional I could find or think of: https://github.com/mattermost/mattermost-webapp/pull/11351

neallred avatar Oct 13 '22 16:10 neallred

Great! Thank you! How do we proceed with all the changes? Should we merge both prs so it's easier to QA or continue in your pr?

javaguirre avatar Oct 13 '22 16:10 javaguirre

my pr is a PR into your branch, so if you like it you can approve and merge it.

neallred avatar Oct 13 '22 16:10 neallred

Thank you! I was reviewing it in the phone and didn't see that 🙂

javaguirre avatar Oct 13 '22 16:10 javaguirre

Successfully triggered e2e testing! https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/250471

mattermod avatar Oct 13 '22 17:10 mattermod

Friendly reminder @DHaussermann :-)

javaguirre avatar Nov 08 '22 08:11 javaguirre