-
Notifications
You must be signed in to change notification settings - Fork 0
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
Enable to delete a product #36
Conversation
✅ Deploy Preview for knights-fecommerce ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for knight-fe ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
da05ef4
to
89e1075
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@UwicyezaG
Please rebase with develop and resolve the conflicts
And increase the test coverage !!
999ac7d
to
cc73f9f
Compare
b1ef24b
to
6304a35
Compare
6c25367
to
3547f19
Compare
64da819
to
b9d81f6
Compare
src/Routes/Router.tsx
Outdated
const isAdmin = decodedToken?.role.toLowerCase() === 'admin'; | ||
const isVendor = decodedToken?.role.toLowerCase() === 'vendor'; | ||
const isBuyer = decodedToken?.role.toLowerCase() === 'buyer'; | ||
const isAdmin = decodedToken?.userType.toLowerCase() === 'admin'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@UwicyezaG You must use the value from 'role' instead of 'userType', as the token does not contain userType.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@UwicyezaG Are we using userType or role ?
@UwicyezaG |
b9d81f6
to
f51257f
Compare
4dddc9f
to
a919981
Compare
a919981
to
aac9a68
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Approved @UwicyezaG !
Enable to delete a product
Enable to delete a product
Enable to delete a product
Enable to delete a product
Enable to delete a product
Description
The seller should be able to delete a specific item in their collection such that they can remove items they no longer deal in or were erroneously added. This functionality ensures that sellers can manage their product listings effectively and maintain an accurate inventory.
Definition of Done:
Screenshots