Closed Haimantika closed 3 months ago
Name | Link |
---|---|
Latest commit | 18f3b1f0df876a42074e47990831f77035ce19db |
Latest deploy log | https://app.netlify.com/sites/sauced-intro/deploys/660076dc764fe800083ceb65 |
Deploy Preview | https://deploy-preview-141--sauced-intro.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Hey @Haimantika. Can you add Closes
next to the issue number in the Related Tickets and Documents section
?
@Haimantika I think your PR here consists the previous commits from your changes on PR #133.
Can you please update your working branch (fix_lesson_5
) by pulliing and merging the main
from upstream?
Thank you.
EDIT: I've updated your branch. You can pull it to your local by running the command git pull
. :)
git pull
Oh thanks, so nothing pending from my end, right?
Hi @Haimantika, Thank you for your PR! 🙌
I left a review here for you to fix before we can approve your PR. I also updated your branch. So, can you please pull your branch before working on the fix?
Let me know when it's fixed and ready for a review. Thank you. 🙂
Accepted changes.
Description
This solves issue https://github.com/open-sauced/intro/issues/140 , where I added a note to fill PR form in lesson 05
What type of PR is this? (check all applicable)
Related Tickets & Documents
Closes https://github.com/open-sauced/intro/issues/140
Mobile & Desktop Screenshots/Recordings
NA
Steps to QA
NA
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?