Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi!
BazaarVoice released v2 of their display integration which brings performance updates and longer support.
See: https://knowledge.bazaarvoice.com/wp-content/conversations/en_US/Display...
For reference, the documentation for the currently used v1: https://knowledge.bazaarvoice.com/wp-content/conversations/en_US/Display...
Comment | File | Size | Author |
---|---|---|---|
#11 | bazaarvoice-bvapi-v2-2959188.patch | 122.04 KB | santoine |
#2 | api-option.patch | 14.87 KB | Jon352 |
Comments
Comment #2
Jon352 CreditAttribution: Jon352 commentedRan into a similar need to use the newer api. When using the newer bv.js you have to make template changes and you don't need to make use of $BV.ui.
I'm attaching a patch I created that should give the user the option to use either bvapi.js or bv.js. It includes some logic that manipulates the templates by adding a apiVersion variable to change the templates. Its not 100% tested but so far it works for me.
I may end up going with a custom lightweight implementation since I really don't need the conversatons api and I may not need the product feed. The bv.js file can be dropped into the theme and use templating to render the product id into the bazaarvoice templates.
Anyways, I hope the patch is helpful to someone.
Comment #3
ahimsauzi@Jon352, I am in the same situation as you and looking to implement a lightweight solution. Can you share what is involved other than adding the library to a theme? I am looking to have reviews on product detail pages and inline reviews on Views blocks. Thanks.
Comment #4
jasonawantWhile this patch applies cleanly, I'm unable to get this patch to work.
I think this Drupal project may need a rewrite as a 3.x version to focus on the Bazaarvoice v2 Display integration. Also, the product feed has been replaced by BV Dynamic Catalog Collection: https://knowledge.bazaarvoice.com/wp-content/conversations/en_US/Collect...
Comment #5
jasonawantFor those using v2, could you collaborate on this #3133902: [Meta]: Plan 8.x-3.x version or 9's 1.0.0 version?
Comment #6
jasonawantI've taken another look at this, and I had to ask BV to enable the v2 for our workbench for this work.
This patch needs code standards check.
Comment #7
santoine CreditAttribution: santoine at Nerdery for Nestle Purina PetCare - United States commentedThis builds off of Jon352 api.option.patch and adds code standards to it.
Comment #8
santoine CreditAttribution: santoine at Nerdery for Nestle Purina PetCare - United States commentedAdd code style changes to John352 patch.
Comment #9
santoine CreditAttribution: santoine at Nerdery for Nestle Purina PetCare - United States commentedThis patch adds code style changes to John352 patch as well as merges https://www.drupal.org/project/bazaarvoice/issues/3124596 into it.
Comment #10
santoine CreditAttribution: santoine at Nerdery for Nestle Purina PetCare - United States commentedComment #11
santoine CreditAttribution: santoine at Nerdery for Nestle Purina PetCare - United States commentedThis patch does the following:
- Add code style changes to John352's patch
- Merge https://www.drupal.org/project/bazaarvoice/issues/3124596
- Merge https://www.drupal.org/project/bazaarvoice/issues/3143372
Comment #12
kamkejj CreditAttribution: kamkejj at Nerdery for Nestle Purina PetCare - United States commentedWe've been using Path #11 on our project and has been working.
Comment #13
kamkejj CreditAttribution: kamkejj at Nerdery for Nestle Purina PetCare - United States commentedThanks Jon352 and santoine. Commioted patch #11 to 8.x-2.x branch.
Comment #15
kamkejj CreditAttribution: kamkejj at Nerdery for Nestle Purina PetCare - United States commented