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

feat(settings): Implement missing calltimer setting #1946

Merged
merged 3 commits into from
Mar 28, 2024

Conversation

Flemmli97
Copy link
Collaborator

What this PR does 📖

  • Implements the missing call timer setting in audio

Which issue(s) this PR fixes 🔨

@github-actions github-actions bot added Don't merge yet DO NOT MERGE Missing dev review Still needs to be reviewed by a dev Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Mar 26, 2024
Copy link
Contributor

github-actions bot commented Mar 26, 2024

UI Automated Test Results Summary for MacOS/Windows

392 tests   365 ✅  2h 33m 49s ⏱️
 28 suites   23 💤
  3 files      2 ❌  2 🔥

For more details on these failures and errors, see this check.

Results for commit 3f9e65a.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Mar 26, 2024

UI Automated Tests execution is complete! You can find the test results report here

@luisecm
Copy link
Contributor

luisecm commented Mar 26, 2024

@Flemmli97 @stavares843 @phillsatellite The automated tests for mac and windows are failing now because currently in dev, Call Timer is set as disabled by default, and on this branch, the Call Timer is enabled by default

I can quickly update the tests to reflect this change once this PR is merged, in case that having this setting enabled by default is fine

On Dev:

image

On this branch:

image

@phillsatellite phillsatellite added QA Tested QA has tested and approved and removed Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Mar 27, 2024
@github-actions github-actions bot added the Failed Automated Test This PR is failing Luis's Appium test and needs revised label Mar 27, 2024
@dariusc93 dariusc93 added Ready to Merge This PR is ready to merge and removed Missing dev review Still needs to be reviewed by a dev Don't merge yet DO NOT MERGE labels Mar 28, 2024
@phillsatellite phillsatellite added Waiting for CI Waiting for at least one CI job to complete before merging and removed QA Tested QA has tested and approved Ready to Merge This PR is ready to merge Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Mar 28, 2024
@github-actions github-actions bot added the Failed Automated Test This PR is failing Luis's Appium test and needs revised label Mar 28, 2024
@stavares843 stavares843 merged commit 761d047 into dev Mar 28, 2024
5 of 7 checks passed
@stavares843 stavares843 deleted the impl-missing-call-timer-setting branch March 28, 2024 18:53
@github-actions github-actions bot added Failed Automated Test This PR is failing Luis's Appium test and needs revised and removed Failed Automated Test This PR is failing Luis's Appium test and needs revised Waiting for CI Waiting for at least one CI job to complete before merging labels Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Failed Automated Test This PR is failing Luis's Appium test and needs revised
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants