Closed CesiumBen closed 2 years ago
CC: @xuelongmu @argallegos
@shehzan10 Thank you for the review. I've addressed your comments, and have a few open for you to double check.
@CesiumBen In the intro paragraphs, maybe indicate that more tutorials will be added over time?
@shehzan10 Pushed some updates addressing your comments, and did some overall cleanup
Be sure up update Cesium for Unreal Samples to VR Tutorials in the License Section of the readme.
"Apache 2.0. Cesium for Unreal VR Tutorials are free to use as starter project for both commercial and non-commercial use."
@LisaBosCesium
In the intro paragraphs, maybe indicate that more tutorials will be added over time?
Appended to the end of the intro
"New tutorials will be added in frequently so check back soon!"
@CesiumBen Looks good to me. Thanks
Summary
This PR updates the Readme for the tutorial release, and addresses changes mentioned here.
Author checklist
Tutorial for N/A
Has the new tutorial been integrated into the main level? N/A
Has a new section of the readme been added for this tutorial? N/A
Have you done a full self-review of the code and blueprints? Yes
Have you tagged a primary reviewer? Yes
Are there any remaining tasks to do or blocking questions to answer before we can merge this? Yes
[x] Links to tutorials are added into readme
Testing plan
N/A
Reminders for reviewers
Thank you for taking the time to review this PR. By approving a PR you are taking as much responsibility for these changes as the author. Please keep these points in mind throughout the review process: