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 admin is not able to create the catalog rule for the configurable product, with SKU condition type #6679

Closed
Sanjaybhattwebkul opened this issue Sep 1, 2022 · 1 comment
Closed

The admin is not able to create the catalog rule for the configurable product, with SKU condition type #6679

Sanjaybhattwebkul opened this issue Sep 1, 2022 · 1 comment
Assignees
@akathuria289
Labels
Admin Admin package related issues. Bug Fixed When bug issue is fixed. Bug Something isn't working.
Milestone
1.4.5

Comments

@Sanjaybhattwebkul
Copy link
Contributor

Bagisto Version: Master

Steps To reproduce--

1. Log in as an admin.
2. Click on the catalog rule inside the Marketing / Promotion.
3. Try to create a catalog rule for any variant of a configurable product.
4. Select SKU on condition type dropdown

Expected result: There should not be any warning message in the condition section if the admin creating the catalog rule for the configurable product,

Actual result: The admin is not able to create the catalog rule for the configurable product, with SKU filter. Please find the screenshot Here
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Something isn't working. Admin Admin package related issues. labels Sep 1, 2022
@Sanjaybhattwebkul Sanjaybhattwebkul added this to the 1.4.5 milestone Sep 1, 2022
akathuria289 pushed a commit to akathuria289/bagisto that referenced this issue Sep 1, 2022
Fixed issue bagisto#6679
e18b6a4
@akathuria289 akathuria289 mentioned this issue Sep 1, 2022
Merged
@akathuria289 akathuria289 added the Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. label Sep 1, 2022
jitendra-webkul added a commit that referenced this issue Sep 6, 2022
@jitendra-webkul
Merge pull request #6682 from akathuria289/issue_#667
ed0e012 
Fixed issue #6679
@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.

@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Fixed When bug issue is fixed. and removed Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. labels Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@akathuria289 akathuria289

Labels
Admin Admin package related issues. Bug Fixed When bug issue is fixed. Bug Something isn't working.
Projects
None yet
Milestone
1.4.5
Development

No branches or pull requests
2 participants
@Sanjaybhattwebkul @akathuria289