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.

Sign up for GitHub

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

Jump to bottom

The delete icon should not be visible when the user is adding only one product in the quote item. #447

Closed
Sanjaybhattwebkul opened this issue Oct 7, 2021 · 3 comments
Closed

The delete icon should not be visible when the user is adding only one product in the quote item. #447

Sanjaybhattwebkul opened this issue Oct 7, 2021 · 3 comments
Labels
Done Enhancement

Comments

@Sanjaybhattwebkul
Copy link
Contributor

CRM version: V1.1.3

Steps To reproduce--

1. Log in as Admin/User.
2. Click on the Quote tab.
3. Click on create Quote button in the top-right corner.
4. Reach-in quote item.

Actual result: The delete icon is visible when the user is not adding more than one product in the quote item. Please Find the blew screenshot.

Expected result: The delete icon should not be visible when the user is adding only one product in the quote item.

Quote-bug-1
jitendra-webkul added a commit to jitendra-webkul/bagisto-crm that referenced this issue Oct 18, 2021
@jitendra-webkul
Issue krayin#447 fixed
b41ff41
@jitendra-webkul jitendra-webkul mentioned this issue Oct 18, 2021
Merged
jitendra-webkul added a commit that referenced this issue Oct 18, 2021
@jitendra-webkul
Merge pull request #523 from jitendra-webkul/master
e123c02 
Issue #447 fixed
@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
When a user is trying to add multiple products, then the delete option should be present there.
And if the user is adding only one product on the quote item, then no need for a delete icon. because one item is mandatory.

Quote-bug

jitendra-webkul added a commit to jitendra-webkul/bagisto-crm that referenced this issue Oct 19, 2021
@jitendra-webkul
Issue krayin#447 fixed
0c821fc
@jitendra-webkul jitendra-webkul mentioned this issue Oct 19, 2021
Merged
jitendra-webkul added a commit that referenced this issue Oct 19, 2021
@jitendra-webkul
Merge pull request #529 from jitendra-webkul/master
462d6dc 
Issue #447 fixed
@jitendra-webkul
Copy link
Member

Check now

@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
Thanks, now this issue is fixed, ao I am closing this issue.
And I have attached a screenshot for future reference.

fixed-1

Fixed-2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Done Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@saurav-webkul @jitendra-webkul @devansh-webkul @Sanjaybhattwebkul