Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update link to MRTK folder in readme #41

Merged
merged 3 commits into from
Nov 4, 2021

Conversation

keveleigh
Copy link
Contributor

@keveleigh keveleigh commented Nov 3, 2021

  • The link to /Assets/SimpleWebXR/Scripts/MRTK-Providers didn't work for me anymore. It seems the new folder is /SimpleWebXR-Demo/Assets/SimpleWebXR/Scripts/MRTK, but let me know if this is also incorrect!
  • Removed some locale-specific portions of URLs, to let the user's browser dictate which locale to use
  • Also added .vscode to the .gitignore

Happy to revert any parts of this or split things out if requested! Didn't want to overload the repo with a bunch of separate PRs, and this was all constrained to one main file.

@Rufus31415 Rufus31415 merged commit a906041 into Rufus31415:master Nov 4, 2021
@Rufus31415
Copy link
Owner

Thanks for this review !

@keveleigh keveleigh deleted the update-readme branch November 4, 2021 18:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants