Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(call): retry connection instead of wait if not made #797

Merged
merged 1 commit into from
Oct 30, 2024
Merged

fix(call): retry connection instead of wait if not made #797

merged 1 commit into from
Oct 30, 2024

Conversation

Flemmli97
Copy link
Contributor

What this PR does 📖

  • Retry call invitation connection instead of just waiting. Also reduce retry wait time

Which issue(s) this PR fixes 🔨

  • Potentially?: Feedback for no response, and when people joined the call maybe is not correct when we have a lot of people in the call (BUG)

@github-actions github-actions bot added the Missing dev review Two dev reviews are required on PR label Oct 30, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 30, 2024 •
edited
Loading

Download the app installers for this pull request:

@github-actions GitHub Actions
Copy link

Automated tests execution is complete! You can find the Playwright test report here and the Allure Test Report here

@phillsatellite phillsatellite added the QA Approved PR has been tested by QA Team label Oct 30, 2024
@stavares843 stavares843 merged commit 6a4d27d into dev Oct 30, 2024
12 checks passed
@stavares843 stavares843 deleted the call-retry branch October 30, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phillsatellite phillsatellite phillsatellite approved these changes

@stavares843 stavares843 stavares843 approved these changes

Assignees

@Flemmli97 Flemmli97

Labels
Missing dev review Two dev reviews are required on PR QA Approved PR has been tested by QA Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Flemmli97 @stavares843 @phillsatellite