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

Add Configurations and frontend changes #7468

Closed
wants to merge 3 commits into from

Conversation

vijay-webkul
Copy link
Contributor

Issue Reference

#7410
-> Add new feature in configuration and show on the frontend page

@Sanjaybhattwebkul Sanjaybhattwebkul added the QA Passed When QA pass the pull request. label Apr 5, 2023
@Sanjaybhattwebkul
Copy link
Contributor

@vijay-webkul
This is working fine.

@Amitk-webkul
Copy link
Collaborator

@vijay-webkul
now It is working fine. Here

packages/Webkul/Admin/src/Config/system.php Outdated Show resolved Hide resolved
@php
$product = $item->product;
@php
$productCount = core()->getConfigData('catalog.products.product-view-page.no-of-cross-sells-product_productpage');
Copy link
Member

Choose a reason for hiding this comment

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

correct the alignment and use snake casing in config field.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

resolved

@Sanjaybhattwebkul Sanjaybhattwebkul added Change Request if need any changes on PR and removed QA Passed When QA pass the pull request. labels Apr 5, 2023
@Sanjaybhattwebkul Sanjaybhattwebkul added QA Passed When QA pass the pull request. and removed Change Request if need any changes on PR labels Apr 5, 2023
Copy link
Member

@devansh-webkul devansh-webkul left a comment

Choose a reason for hiding this comment

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

Check the comments.

@devansh-webkul devansh-webkul added Change Request if need any changes on PR and removed QA Passed When QA pass the pull request. labels Apr 5, 2023
@vijay-webkul
Copy link
Contributor Author

working on this

@Sanjaybhattwebkul
Copy link
Contributor

@vijay-webkul
Any update ?

@devansh-webkul devansh-webkul added Conflicts When conflicts present in pull request. and removed Change Request if need any changes on PR Conflicts When conflicts present in pull request. labels Apr 21, 2023
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.

None yet

5 participants