Skip to content

[Template]added new input package as template dependancy #7419

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Verasl
Copy link
Contributor

@Verasl Verasl commented May 10, 2022

Purpose of this PR

New input system wasn't added when previous work done to support it in the templates, this PR fixes that


Testing status

Loaded the template and observed no errors


Comments to reviewers

Need QA to check platforms and controls still work

@Verasl Verasl requested a review from a team as a code owner May 10, 2022 13:49
@Verasl Verasl requested a review from a team May 10, 2022 13:49
@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page).
See the PR template for more information.
Thank you!

@ernestasKupciunas ernestasKupciunas requested review from ernestasKupciunas and removed request for a team May 18, 2022 05:48
Copy link

@ernestasKupciunas ernestasKupciunas left a comment

Choose a reason for hiding this comment

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

Controls for mobiles are not working. Works fine on Standalone MacOS.

Tested with:
Controls are not working:
VLNQA00268, Samsung Galaxy S10+, 10.0.0, Exynos 9 9820, Mali-G76
VLNQA00279 - Samsung Galaxy S10+ USA (SM-G975U), CPU: Snapdragon 855 SM8150, GPU: Adreno 640, OS: 9.0.0
iPhone 12Pro, SoC: A14, iOS: 14.1

Controls work fine:
MacBook Pro (16-inch, 2021), CPU: Apple M1 Max, Memory: 64 GB, macOS: Monterey 12.1

@memphis88 memphis88 requested a review from a team as a code owner August 13, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants