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

Wait for XR before play #48

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Wait for XR before play #48

merged 1 commit into from
Aug 6, 2024

Conversation

lmidang
Copy link

@lmidang lmidang commented Aug 6, 2024

Description:
Fix for this issue

Testing

Screen_Recording_20240806_110525_Chrome.mp4

@cmbartschat
Copy link
Collaborator

Can you use the same commit message as db16b54 so it's easier to track?

@cmbartschat
Copy link
Collaborator

If we revert to the aframe 1.5.0 commit and apply all the picks from before, are there any missing that this branch doesn't have?

@lmidang lmidang changed the title Fixed issue with loading XR with aframe Wait for XR before play Aug 6, 2024
@lmidang
Copy link
Author

lmidang commented Aug 6, 2024

If we revert to the aframe 1.5.0 commit and apply all the picks from before, are there any missing that this branch doesn't have?

which 1.5.0 commit?

@cmbartschat
Copy link
Collaborator

Copy link

@yuyansong-8thwall yuyansong-8thwall left a comment

Choose a reason for hiding this comment

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

LGTM

@lmidang
Copy link
Author

lmidang commented Aug 6, 2024

The commit corresponding with https://github.com/aframevr/aframe/releases/tag/v1.5.0 -

aframevr@7856a05

I think this branch was pulled after the commit and added some changes, so I don't think it's missing anything from 1.5.0

@lmidang lmidang merged commit 409dd76 into master Aug 6, 2024
1 check failed
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.

3 participants