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

docs: small changes #950

Merged
merged 2 commits into from
Jun 17, 2023
Merged

docs: small changes #950

merged 2 commits into from
Jun 17, 2023

Conversation

taku-nm
Copy link
Contributor

@taku-nm taku-nm commented Jun 16, 2023

merging footnote 1 and 4; some more clarity for step 6

merging footnote 1 and 4; some more clarity for step 6
@taku-nm taku-nm changed the title tiny Update 2_1_patching.md Update 2_1_patching.md Jun 16, 2023
and hopefully no semantics error lol
@taku-nm taku-nm changed the title Update 2_1_patching.md docs: small changes Jun 16, 2023

[^1]: You may be prompted to select an app from storage in case you have no root access on your device. In this case, select the APK file from your device storage.[^3]
[^1]: Non-root users may be prompted to select an APK from storage, in which case you have to source the APK file yourself. ReVanced does not provide any APK files.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The information about having to source APK files yourself should be added to the step where the user needs to select an app. When they already selected one and are already in the installation screen, what use has the footnote about them having to source the APK file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that not currently the case? Footnote 1 mentions that and is attached to step 3, where the user needs to select an app. That would also be the exact point where they might be prompted about non-root installation, giving an incentive to read this footnote, to realize that they have to select from storage and have to source it themselves.

@oSumAtrIX oSumAtrIX merged commit af054fb into ReVanced:dev Jun 17, 2023
1 check passed
ponces pushed a commit to ponces/revanced-manager that referenced this pull request Jun 22, 2023
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.

None yet

2 participants