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

Add improved documentation for installation and getting started #5545

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lucyjin3
Copy link

@lucyjin3 lucyjin3 commented Feb 12, 2024

Description (required)

Updated the README file to improve clarity and provide better guidance for users installing, getting started with, and using the Commons Android app. Added sections for installation instructions, getting started tips, and usage examples to enhance the user experience and facilitate contributions to the project.

Installation Instructions:

Check if the installation instructions are clear and comprehensive, covering the steps required to install and set up the app on an Android device or emulator.
Verify if there are any dependencies or prerequisites mentioned, such as minimum Android version or additional libraries.

Getting Started Section:

Assess whether there's a clear "Getting Started" section that helps new users understand how to use the app after installation.
Look for information on basic app functionality, navigation, and any initial setup steps required.

Usage Examples:

Look for usage examples or tutorials that demonstrate how to use key features of the Commons Android app.
Evaluate if there are any sample workflows or scenarios provided to help users understand the app's capabilities.

Tests performed (required)

Tested ProdDebug build variant on Pixel 5 device with API level 30.

Copy link
Member

@domdomegg domdomegg left a comment

Choose a reason for hiding this comment

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

Hi @lucyjin3, thanks for the contribution! There is already user documentation linked to from the README. I'm concerned that this PR might be duplicating some of that, as well as the in-app messaging users will already see. This therefore potentially increases the scope of what needs to be maintained for limited user benefit.

Could you clarify what you expect adding this documentation here will do over and above the existing documentation?

@@ -1 +0,0 @@
Include release notes with alpha releases
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure what removing this file is doing, or whether it's related to this PR?

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