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

Added PRODUCT_DETAILS_RESET #161

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Added PRODUCT_DETAILS_RESET #161

wants to merge 5 commits into from

Conversation

zayedibnibrahim
Copy link

When you Edit any product it works good, but if you go back and again try to edit the same product again, in the form it will show you the old data. Thats why Adding PRODUCT_DETAILS_RESET is very much important.

Added  PRODUCT_DETAILS_RESET , because when update any product at ProductEditScreen.js as Admin, if you again try to click Edit the same product it will show you the old product details, not the updated one
Reducer for PRODUCT_DETAILS_RESET
@talmax1124
Copy link

Hello @zayedibnibrahim, could you help me implement add-ons / options into my repo? I saw you did something similar in your repos. If you can help, I would really appreciate it!

@zayedibnibrahim
Copy link
Author

Hello @talmax1124, To implement add-ons functionality you have to use Multi-select component. There is few npm package available for Multi-select. but i have used multi-select component from Ant Design. I found it pretty much easier to implement.
There you go : https://ant.design/components/select/

@talmax1124
Copy link

talmax1124 commented Aug 6, 2021

Hello @talmax1124, To implement add-ons functionality you have to use Multi-select component. There is few npm package available for Multi-select. but i have used multi-select component from Ant Design. I found it pretty much easier to implement.
There you go : https://ant.design/components/select/

@zayedibnibrahim Thanks for that suggestion! Could you help implement that feature into my repo, which is basically the same exact thing as this? Because I have tried it myself before, but I failed and I really need someone to help me out.

@talmax1124
Copy link

@zayedibnibrahim I've tried doing it myself like 3 times and I have had no success. I would really appreciate if you would help me out in this. Thanks!

@zayedibnibrahim
Copy link
Author

@talmax1124 I am still working on the client side. The add-ons currently working only in server side. But I am dealing with a tons of issue on the client side. because it also needs to display the add-ons on the client side.

@talmax1124
Copy link

@zayedibnibrahim Well when you figure it out can you tell me and help add it to mines?

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.

2 participants