-
Notifications
You must be signed in to change notification settings - Fork 380
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
Fixes #1196 ✔️ Added description on dependency download in Android Studio #1214
base: develop
Are you sure you want to change the base?
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.
Can you add a screenshot in the PR comment showing the changes you made. Otherwise, it looks good to me. Thanks for contributing. :)
@tinniaru3005 added the screenshots. Pls review. |
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.
Hey @pratyaksh1610 . Congratulations on making a PR to the project. Thanks for adding the screenshot. The changes are fine. Thanks for contributing. If you have any further doubts you can reach us at zulip chat :)
Why these checks are failing ? some of them, or it requires another person's review |
Well, I don't have much idea regarding that. You can ask your doubts at zulip chat and the admins can help you with that. But for Open Souce Hack it is not necessary to get your PRs merged. :) |
But I think PR is required to be merged as it is part of documentation , and for a project , a clear and concise documentation is much needed. |
@isabelcosta Please review this PR, so that it can be merged. |
@vj-codes Pls review this PR so that it can be merged. |
@vj-codes @vatsalkul Please review this PR so that it can be merged. |
Description
Fixes #1196
Added description on dependency download in Android Studio.
Type of Change:
Checklist: