loader at checkout form
-
@Rahul74 said in loader at checkout form:
@Vaishali-Agarwal For configurable product, as low as price showing 0, its the master version, can u plz check if i'm wrong or right??
-
@Rahul74
Yes, for configurable product price(base product) is $0 , it's an bug and already reported here https://github.com/bagisto/bagisto/issues/3558 , which is fixed and ready to be merge. -
@Vaishali-Agarwal will u plz share me its PR as soon as its fixed? thank u
-
-
@Vaishali-Agarwal thank u so much.
-
@Vaishali-Agarwal one more thing i couldn't able to do is in the checkout form, the last name is required, i want to remove the last name, can u help how to do it?
-
@Rahul74
removing the last name will take a lot of code to be change, instead of this you can also set Last_Name field to be non-required fields, so if user can proceed with adding the lastname -
@Vaishali-Agarwal Hi can u tell me what thise rich snippets are for ?
-
@Rahul74
Rich snippets help Google search engines better comprehend its context for search results display. When you enable the Rich Snippets for products then this employed, search engines are able to display more information about the actual result, including your products ratings, reviews, price etc and increase your website page rank.Thanks
-
@Vaishali-Agarwal i just pulled from the master branch, but don't have any of this 'loading' effect
currently, i using my custom theme but just copied velocity theme with little design change..is there's anything i need to do/copied(which file?) after pulling the latest from the master branch ?
Edit:
I just realise this error on console log "Uncaught (in promise) TypeError: this.$root.showLoader is not a function
at onepage:2890"
What to do ? -
@otheriz
can i access your website url where you have done all these changes? -
@Vaishali-Agarwal Is this the correct solution?? i have removed those two lines of codes indicated with arrow. For loader occuring even entering single digit..
-
Hi @Rahul74
No this is not the solution, as provided in the screenshot, the solution is mention in the PR which is signed with merged now as https://prnt.sc/tp89he
The PR is here https://github.com/bagisto/bagisto/pull/3576/files -
@Vaishali-Agarwal i did this change but its not seemed to fix that,
-
Hope you have run these commands
composer dump-autoload
php artisan config:cache
php artisan vendor:publish --force -
@Vaishali-Agarwal it's same.
-
@Vaishali-Agarwal we don't need to remove any of that?
-
@Rahul74
No, no need to remove this.$root.showLoader();
I also changed my code which is already merged in the master and it's working fine -
-
@Rahul74
now working??