Skip to content
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

Tlf 827 #5

Merged
merged 7 commits into from
Sep 6, 2022
Merged

Tlf 827 #5

merged 7 commits into from
Sep 6, 2022

Conversation

onyebuchi702
Copy link

@onyebuchi702 onyebuchi702 commented Aug 29, 2022

What problem is this solving?

fixing the remove from wishlist message and showed the toast when an item is removed from the wishlist

How it works:

add and remove an item from the wishlist on plp or pdp

How to test it?

Provide sufficient enough information for a non developer to test
Workspace
https://tlf827--thefoschini.myvtex.com/men/accessories?order=OrderByReleaseDateDESC&__device=desktop

Screenshots/Video example usage:

Screenshot 2022-09-02 at 12 40 10

Screenshot 2022-09-02 at 12 40 43

Screenshots/Video of effect on all relevant breakpoints

Describe alternatives you've considered, if any.

Related to / Depends on

none

Coding Standards Checklist: https://tfginfotec.atlassian.net/l/c/60ZyZonH

How does this PR make you feel? 🔗

good

@onyebuchi702
Copy link
Author

@lezegom @giretyu please review

@simmbiote simmbiote merged commit 9271d2f into master Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants