Closed govindumeesala closed 1 month ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
scrollme-ecommerce-website | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Oct 7, 2024 6:55am |
@thakuratul2 @saurabhbakolia please consider adding level3 or level2 label to the pr. I worked really hard to implement and come up with the logic.
Hey @govindumeesala can you create a menu like this? A dropdown will open when the user clicks on the profile menu.
Make sure you create a PR to develop
the branch. If you have any other better idea, please suggest.
Name | Link |
---|---|
Latest commit | d9406378fa3866511b47cb66fdd6405a0091d63c |
Latest deploy log | https://app.netlify.com/sites/scrollme-ecommerce/deploys/670385b13db019000847b6b2 |
Deploy Preview | https://deploy-preview-125--scrollme-ecommerce.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 @govindumeesala can you create a menu like this? A dropdown will open when the user clicks on the profile menu.
Make sure you create a PR to
develop
the branch. If you have any other better idea, please suggest.
Sure @saurabhbakolia
Can you please consider merging this pr.
I will raise new issue profile pop-up and work on it.
If you check this once and merge , it means a lot to me.
@govindumeesala you create new issue when we review it will be merge. So meanwhile you start working
@govindumeesala you create new issue when we review it will be merge. So meanwhile you start working
ok thank you
@govindumeesala why this branch having code of contact us page?
@govindumeesala why this branch having code of contact us page?
I previously worked on contact us page and faq . I thought after merging the that pr. This will be merged, so I created a new branch (bugLogout) #125 from that branch (featReachoutUS) #90 @thakuratul2 you said all fine with that pr #90 , but it didnt get merged yet.
@govindumeesala no why you create the branch from that one you need to create new branch always from develop if they updated or not due to this your previous commits also comes here we cannot be merge this
@thakuratul2 From next time I will definitely do it from develop branch. please check once, There are no conflicts. All working fine. First merge #90 Then merge #125 I will for sure create a branch from develop from next time Please consider once, since, everything is working fine.
Thankyou
@govindumeesala so please close the your last pr then I will merge it
Pull Request Template
Title: Logout functionality
Description
Added logout functionality Updated both frontend and backend
Related Issue
`Fixes #102
Changes Made
Checklist
Screenshots (if applicable)
@saurabhbakolia @thakuratul2 added logout functionality