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

Expanded the explanation of the import process #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Lunerwalker2
Copy link
Contributor

This has stemmed from a few occurrences of people asking about issues with importing the RoadRunner quickstart in the FTC Discord. Most of them are caused by importing the outer folder of the extracted zip, or by not accepting the SDK Licenses. The existing explanation for the process doesn't specifically cover these potential issues.

This is a quick type-up that attempts to expand the steps to prevent a user from repeating one of the mistakes I listed above. I would like to add some images, but I'm not at home right now and I am unable to use Android Studio. If you think that some images would be good, then I can certainly do that later today.

  • Added a warning to not import the outer folder of the extraction
  • Added steps for how to accept the SDK Licenses if needed

- Added a warning to not import the outer folder of the extraction
- Added steps for how to accept the SDK Licenses if needed
- Might add images if needed, but for now it's probably fine
@netlify
Copy link

netlify bot commented Oct 25, 2021

✔️ Deploy Preview for eloquent-panini-5c807a ready!

🔨 Explore the source changes: 14f6d1e

🔍 Inspect the deploy log: https://app.netlify.com/sites/eloquent-panini-5c807a/deploys/6176b1ceb5da2900087e12ec

😎 Browse the preview: https://deploy-preview-23--eloquent-panini-5c807a.netlify.app

@NoahBres
Copy link
Owner

Honestly I have zero clue why I never merged this

@Lunerwalker2
Copy link
Contributor Author

I can close it if you want lol, unless it's still useful.

@NoahBres
Copy link
Owner

Nah I'm keeping it open to revisit again in the future

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.

2 participants