In Getting Stared guide, clarify Android project integration #2166
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jamesonwilliams
approved these changes
Aug 8, 2020
Contributor
jamesonwilliams
left a comment
jamesonwilliams
left a comment
There was a problem hiding this comment.
Looks good, thanks @KristinaPeterson.
| ## Configure Amplify and DataStore | ||
|
|
||
| First, we'll add the DataStore plugin and configure Amplify by creating an Application class and overriding the `onCreate()` method. | ||
| 1. Navigate to the **example.todo** folder located at **ToDo** > **app** > **src** > **main** > **java/kotlin** > **com** > **example.todo** |
Contributor
There was a problem hiding this comment.
The . in example.todo:
This probably is a valid instruction, if the user is navigating a UI pane in Android Studio. Android Studio will break from listing directory structure, and instead list Java packages in some cases. In Android projects, directory structure and java package identifiers match, except for / being written as ., instead.
Not sure if this is worth clarifying, or how best to do it (without getting into trivialities.)
TLDR: example.todo is actually a directory path itself, **example** > **todo**.
jamesonwilliams
changed the title
jamesonwilliams
changed the title
jamesonwilliams
added
amplify/android
getting-started
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@mohitsriv Minor updates to clarify instructions for a beginner/first-time customer.
Issue #, if available:
Description of changes:
In Configure Amplify and DataStore:
Clarification on step #1
Added new step between #5 and #6 with information to add to the manifest node
Added more detail to step #7
In Query Todos:
Step #4 small language edit
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.