-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update the readme me file #336
Comments
@HardikGoyal2003 PR made to this issue, please review it and suggest changes, if required. |
I am interested in solving this issue, please assign it to me. @HardikGoyal2003 |
@Flying-Pegasus Per the guidance at https://github.com/oppia/oppia/wiki/Contributing-code-to-Oppia#choosing-a-good-first-issue, please provide an explanation of what your PR will do (with names of files you're changing, what you plan to change in each file, etc.). If it looks good, we can assign you to this issue. Please also follow the other instructions on that wiki page if you have not yet done so. Thanks! |
hi @HardikGoyal2003 , |
@faryalRiz1 Per the guidance at https://github.com/oppia/oppia/wiki/Contributing-code-to-Oppia#choosing-a-good-first-issue, please provide an explanation of what your PR will do (with names of files you're changing, what you plan to change in each file, etc.). If it looks good, we can assign you to this issue. Please also follow the other instructions on that wiki page if you have not yet done so. Thanks! |
Proposed Update for the README.md FileI would like to update the README.md file to provide clear and detailed instructions on where and how to add the repository name in the constant.js file. Below is the breakdown of the planned changes: Files to be Updated
Proposed Changes
Installing the Bot on a Repository
4.Save the changes to Why These Changes Are Necessary
Next StepsHey, @HardikGoyal2003 if this issue is assigned to me, I will:1.Implement the above changes in the |
@Flying-Pegasus This seems fine to me. Thanks for taking a look, I've assigned you -- feel free to submit a PR! |
@seanlip @HardikGoyal2003 I have updated the README.md as I stated above and created a PR. Please review and merge it. |
Is your feature request related to a problem? Please describe.
Readme file's this heading doesn't provide a clear description of where to add the repo name in the
constant.js
file.Describe the solution you'd like
Add the description
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: