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.

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

fix(Call): Avoid incoming call from the same call if it is running #748

Merged
merged 6 commits into from
Oct 23, 2024

Conversation

lgmarchi
Copy link
Collaborator

What this PR does 📖

  • When you are in call, you are not receiving incoming from the same call, just if other call chat starts.

  • Fixed timestamp on controls (Now you can go to other pages and back, it will calculate again correct call time)

image
  • Added indicator that there is a call in progress for that specific chat
image
  • If you are in a call, app will block you to do new calls before finish current one, and you will receive a notification
image

Which issue(s) this PR fixes 🔨

  • Resolve #

Special notes for reviewers 🗒️

Additional comments 🎤

@lgmarchi lgmarchi self-assigned this Oct 22, 2024
@github-actions github-actions bot added the Missing dev review Two dev reviews are required on PR label Oct 22, 2024
@lgmarchi lgmarchi changed the title fix(Call): Fix incoming call during a call fix(Call): Avoid incoming call from the same call if it is running Oct 22, 2024
Copy link

github-actions bot commented Oct 22, 2024

Download the app installers for this pull request:

Copy link

github-actions bot commented Oct 22, 2024

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 23, 2024
@lgmarchi lgmarchi merged commit 2162589 into dev Oct 23, 2024
10 checks passed
@lgmarchi lgmarchi deleted the fix-incoming-call-during-a-call branch October 23, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing dev review Two dev reviews are required on PR QA Approved PR has been tested by QA Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants