Skip to content

Update the plugin to Godot v2 Android plugin - #43

Merged
m4gr3d merged 1 commit into
GodotVR:masterfrom
m4gr3d:refactor_to_v2_android_plugin
Sep 7, 2023
Merged

Update the plugin to Godot v2 Android plugin#43
m4gr3d merged 1 commit into
GodotVR:masterfrom
m4gr3d:refactor_to_v2_android_plugin

Conversation

@m4gr3d

@m4gr3d m4gr3d commented Aug 18, 2023

Copy link
Copy Markdown
Collaborator

Update based on godotengine/godot#80740.

Will be removed from draft once that PR is landed.

@m4gr3d m4gr3d added the enhancement New feature or request label Aug 18, 2023
@m4gr3d m4gr3d added this to the 2.0.0 milestone Aug 18, 2023
@m4gr3d
m4gr3d requested a review from BastiaanOlij August 18, 2023 20:01
@BastiaanOlij

Copy link
Copy Markdown
Member

Cool, so we'll also need to do another test release once merged. :)

@BastiaanOlij

Copy link
Copy Markdown
Member

@m4gr3d the upstream PR has been merged into master, is there anything left to do here or should we take this out of draft?

@m4gr3d
m4gr3d force-pushed the refactor_to_v2_android_plugin branch from 44c4458 to 87a7aca Compare September 6, 2023 06:30
@m4gr3d
m4gr3d marked this pull request as ready for review September 6, 2023 06:32
@m4gr3d

m4gr3d commented Sep 6, 2023

Copy link
Copy Markdown
Collaborator Author

@m4gr3d the upstream PR has been merged into master, is there anything left to do here or should we take this out of draft?

Ready for review!

@BastiaanOlij BastiaanOlij left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested both on Quest Pro and PICO 4, looks like it all works :)

@m4gr3d
m4gr3d merged commit 8a99591 into GodotVR:master Sep 7, 2023
@m4gr3d
m4gr3d deleted the refactor_to_v2_android_plugin branch September 7, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants