-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Create update_issue_#11009 #11044
Create update_issue_#11009 #11044
Conversation
Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. |
Code Climate has analyzed commit e6b9862 and detected 0 issues on this pull request. View more on Code Climate. |
@TildaDares can u review on this please |
Hello @ayish1998, you have to mention the issue number which is linked with this pull request in place of #0000. Thank you! I am attaching a sample screenshot below, you can refer to it |
This pull request generated screenshots of many common pages in the running app. You should be able to download and view them here: |
hello @Somya-Singhal thank you |
Hi @ayish1998, can you add the issue link to the description above? The issue number is at the top of the page(next to the issue title) you were assigned and starts with |
Hi @ayish1998, you are not supposed to replace the entire content of the file. The lines in red should be removed and the ones in green should be added. Can you take a look at the issue again and read the description? Thanks!! |
@TildaDares that's exactly what I actually did. I removed the ones with red highlights and replaced them with the green. |
@ayish1998 Actually it looks like you copied everything. |
Hi @ayish1998, do you need help with this? |
@TildaDares I think so because I don't really know if the instruction I followed is not what is expected. |
|
Hi @ayish1998, any updates on this? |
|
@TildaDares sorry for the late response, I was having exams. Yes, I think I need help. |
Hi @ayish1998, just follow the steps I outlined here #11044 (comment). |
@TildaDares Please on the project timeline, do you have a specific comment on this? |
@ayish1998 You’re expected to break down the project tasks into smaller parts for each week. |
@TildaDares Please do that means I will choose my timing? |
Yes you choose your timing. |
Thank you @TildaDares |
Fixes #0000
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment below