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

update(KeyBinds): Create shortcut to open uplink when it is minimized #1735

Merged
merged 18 commits into from
Jan 24, 2024

Conversation

lgmarchi
Copy link
Contributor

What this PR does 📖

1. Create shortcut to open uplink when it is minimized

Screen.Recording.2024-01-19.at.10.19.39.mov

Which issue(s) this PR fixes 🔨

  • Resolve #

Special notes for reviewers 🗒️

Additional comments 🎤

@lgmarchi lgmarchi self-assigned this Jan 19, 2024
@github-actions github-actions bot added Don't merge yet DO NOT MERGE Missing dev review Still needs to be reviewed by a dev labels Jan 19, 2024
Copy link
Contributor

github-actions bot commented Jan 19, 2024

UI Automated Test Results Summary for MacOS/Windows

500 tests   448 ✅  2h 32m 45s ⏱️
 42 suites   52 💤
  3 files      0 ❌

Results for commit 9bab3d3.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Jan 19, 2024

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

@phillsatellite
Copy link
Contributor

@lgmarchi Hey bro, so on Mac the maximize is working great but on windows I wasnt able to get it to work

20240122-1741-14.3311313.mp4

@phillsatellite phillsatellite added the Changes requested When other dev or QA request a change label Jan 22, 2024
@lgmarchi lgmarchi added Ready for testing Ready for QA to test and removed Changes requested When other dev or QA request a change labels Jan 23, 2024
@github-actions github-actions bot added the Failed Automated Test This PR is failing Luis's Appium test and needs revised label Jan 23, 2024
@phillsatellite
Copy link
Contributor

Lucas is taking a look into an issue I DM'd him about, if you are clicked into another app like Chrome and then you use the shortcut to maximize Uplink you will see Uplink flash open for a minute but it will get hidden behind Chrome

Screen_Recording_2024-01-23_at_11.46.03_AM.mov

@phillsatellite phillsatellite added Changes requested When other dev or QA request a change and removed Ready for testing Ready for QA to test labels Jan 23, 2024
@github-actions github-actions bot removed the Failed Automated Test This PR is failing Luis's Appium test and needs revised label Jan 23, 2024
@phillsatellite phillsatellite removed the Changes requested When other dev or QA request a change label Jan 23, 2024
@phillsatellite phillsatellite added the Waiting for CI Waiting for at least one CI job to complete before merging label Jan 23, 2024
@luisecm luisecm removed the Waiting for CI Waiting for at least one CI job to complete before merging label Jan 23, 2024
@github-actions github-actions bot added linter failing Cargo Workflow (linter) failed on this PR Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Jan 23, 2024
@github-actions github-actions bot removed the linter failing Cargo Workflow (linter) failed on this PR label Jan 23, 2024
@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 Jan 23, 2024
Copy link
Contributor

@InfamousVague InfamousVague left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small change requested then it should be good to go!

common/locales/en-US/main.ftl Outdated Show resolved Hide resolved
@lgmarchi lgmarchi merged commit 9212f06 into dev Jan 24, 2024
7 of 9 checks passed
@lgmarchi lgmarchi deleted the create-shortcut-to-open-uplink-when-it-is-minimize branch January 24, 2024 13:41
@github-actions github-actions bot removed QA Tested QA has tested and approved Missing dev review Still needs to be reviewed by a dev Don't merge yet DO NOT MERGE labels Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants