-
Notifications
You must be signed in to change notification settings - Fork 32
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
Venkata/display scene from mobile scene package #127
Venkata/display scene from mobile scene package #127
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @vbharath98, very impressive work here! Sample works as expected, just a few comments to help optimize the code and use patterns similar to other samples.
Once you have the changes in we can take a look at the Github checker to see if any warnings pop up, and we could resolve them in the next review.
.../src/main/java/com/esri/arcgismaps/sample/displayscenefrommobilescenepackage/MainActivity.kt
Outdated
Show resolved
Hide resolved
...a/com/esri/arcgismaps/sample/displayscenefrommobilescenepackage/components/SceneViewModel.kt
Outdated
Show resolved
Hide resolved
...a/com/esri/arcgismaps/sample/displayscenefrommobilescenepackage/components/SceneViewModel.kt
Outdated
Show resolved
Hide resolved
...a/com/esri/arcgismaps/sample/displayscenefrommobilescenepackage/components/SceneViewModel.kt
Outdated
Show resolved
Hide resolved
...ain/java/com/esri/arcgismaps/sample/displayscenefrommobilescenepackage/screens/MainScreen.kt
Outdated
Show resolved
Hide resolved
...ain/java/com/esri/arcgismaps/sample/displayscenefrommobilescenepackage/screens/MainScreen.kt
Show resolved
Hide resolved
This reverts commit 79ba36a.
…ps://github.com/Esri/arcgis-maps-sdk-kotlin-samples into venkata/display_scene_from_mobile_scene_package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks for adding in the changes @vbharath98 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work @vbharath98
Looks good 👍🏼
Description
PR to add a new Kotlin sample "Display a scene from a mobile scene package" in
Edit and Manage Data
category.Links and Data
Task Link:
runtime/kotlin/issues/2570
What To Review
README.md
andREADME.metadata.json
filesHow to Test
Run the sample on the sample viewer or the repo.