search

Anita77777 / capstone-project

capstone-project-plum.vercel.app
0 stars 0 forks source link

Add Delete Button #14

Closed Anita77777 closed 2 years ago

Anita77777 commented 2 years ago

https://github.com/Anita77777/capstone-project/issues/11 Create delete button for card component

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
capstone-project ✅ Ready (Inspect) Visit Preview May 24, 2022 at 7:52AM (UTC)
doemser commented 2 years ago

In Slack you wrote: "Thanks for the review. The bookmarks are another branch wich were already approved. Should I just apply the changes you requested regarding the bookmarks on this branch?"

I see that there is a branch that was about the bookmark feature and was already merged. The changes I requested are regarding another feature, so you could go for a refactoring of your bookmark functionality in a separate PR to keep working on one feature at once.

As long as you promise to refactor it and change the function naming, I will approve your PR here and wait for your next one where you refactor it :P

Anita77777 commented 2 years ago

I changed the function name and will immediately work on the refactoring of the bookmark buttons