Skip to content
This repository has been archived by the owner on Oct 29, 2024. It is now read-only.

Select option for category in create product screen #97

Conversation

s-vamshi
Copy link
Contributor

@s-vamshi s-vamshi commented Oct 16, 2022

This PR resolves #91.
Added pre - defined options to category field in create product screen and product edit screen.

}}
>
<option>--Select One--</option>
<option value="vegetables">Vegetables</option>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These fields should be dynamic. We have categories data stored in the DB. It will be great if you use those data to render categories.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure will work on it.

@s-vamshi
Copy link
Contributor Author

@roopeshsn I have made changes. Can you please check?

@roopeshsn
Copy link
Owner

Could you share a screenshot? "--Select one--" is also an option right? What if the admin chooses "--Select One--" and proceeds to create a product?

@roopeshsn
Copy link
Owner

Have you formatted the code?

@s-vamshi
Copy link
Contributor Author

Could you share a screenshot? "--Select one--" is also an option right? What if the admin chooses "--Select One--" and proceeds to create a product?

yes i have made changes so when admin chooses "--Select one--" option this message will be shown
image

@s-vamshi
Copy link
Contributor Author

Have you formatted the code?

Done

Copy link
Owner

@roopeshsn roopeshsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @s-vamshi

@roopeshsn roopeshsn merged commit d79a958 into roopeshsn:master Oct 16, 2022
@roopeshsn roopeshsn added the hacktoberfest-accepted Accepted PR for hactoberfest label Oct 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
hacktoberfest-accepted Accepted PR for hactoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set pre-defined options in Category while creating new product
2 participants