-
Notifications
You must be signed in to change notification settings - Fork 65
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
MM-44946: Fix: Calls: start trial path from Start Call button not working #105
Conversation
I'm not able to test this, unfortunately (split.io). So I'll ask for a review now and hope something changes tomorrow, but probably not in the time we have. (Do we even have time? I'm unsure whether we can fix this before release.) |
We don't have time to include it in 7.0/cloud build but what we can probably do, similarly to other bugs (e.g. the thread autofollow one) we can cut a new release later today, push it to the marketplace and force an update on cloud (through split). That should give us what we want without going through pre-packaging. |
@streamer45 Ok, that sounds good, let's do that. 👍 |
@cpoile Do you want to test this before or after merging? |
@streamer45 Do we need to test anything else in this batch? If not, I can cut it from this hash. |
@cpoile I think this is the only PR requiring an actual Cloud instance to be verified so I'd say this is it. I don't have to merge anything else for v0.6.1 (or whatever it's going to be). |
/update-branch |
Huh, guess that doesn't work because we use |
Summary
Ticket Link