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 issue preventing enabling the remote button for Android/iOS #94425

Merged

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Jul 16, 2024

#92032 updated the logic to enable / disable the remote debug button, and in doing so added a can_export check.

However, no events / notifications are dispatched when the value of the can_export check changes, which in turn prevents the logic used to enable / disable the remote debug button from running again.

The fix consists then in removing the can_export check, so that the remote debug button shows as enabled when a preset is present and is runnable.

Fixes #93531

godotengine#92032 updated the logic to enable / disable the remote debug button, and in doing so added a `can_export` check.

However, no events / notifications are dispatched when the value of the `can_export` check changes, which in turn prevents the logic used to enable / disable the remote debug button from running again.

The fix consists then in removing the `can_export` check, so that the remote debug button shows as `enabled` when a preset is present and is runnable.
Copy link
Contributor

@dsnopek dsnopek left a comment

Choose a reason for hiding this comment

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

Thanks!

This fixes the issue in my testing. And, it doesn't cause the return of #91981 which is what PR #92032 was meant to fix.

The code looks good to me too :-)

@akien-mga akien-mga merged commit 7b12dbd into godotengine:master Jul 17, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks!

@m4gr3d m4gr3d deleted the fix_remote_android_button_not_enabling branch July 17, 2024 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Release Blocker
Development

Successfully merging this pull request may close these issues.

Remote debug button for Android not enabling automatically after installing build template for Gradle build
3 participants