reviewscouk/magento2-reviews

Reviews.co.uk Magento 2 Integration

magento2-reviews's Issues

Feature request: Add GTIN to product catalogue feed

In Reviews.io the ability exists to add product feeds. Product GTINs are necessary to connect with services, but the Magento module does not allow GTIN to be outputted in the feed.

Suggestion: in https://github.com/Reviewscouk/magento2-reviews/blob/master/Controller/Index/Feed.php add a GTIN property.

• Add store config option to select the product attribute that represents the GTIN
• If this product attribute is selected, add GTIN property to the feed
• If product attribute is not selected (left blank), skip GTIN property

feat: move api calls to async

Magento supports an async API for sending messages to external services (like reviews.io).

• This plugin should be reworked to use the message queue to send orders to reviews.io.

Otherwise, business users (and other external processes that create shipments) currently have to wait to save shipments in order to send orders to reviews.io.

Content Security Policy Error

In Magento 2.3.5-p1 live website I am getting the CSP Error. I have used the latest release and still I get this error. Please try to fix this error. If you need any more details please feel free to ask.

Issue with Simple Products with Custom Options

Hi,

I've had this issue a couple of years and never got around to looking into it.

Scenario;

1. We sell simple products with custom options using the option to append extra SKU/string to the original simple sku which is default functionality.
2. We use your module to send the product feed to the reviews.io system/website.
3. We use your module to send notification of dispatch for a review to be cued on your website.

The issue I noticed;

1. When I use widgets on my website to show product reviews the product images don't show in the widget.
2. Product images don't show on your website against the product review.

Cause;
When the module sends the data for review it sends the long SKU which is generated from the options selected which is stored in the sales_order_items table.

Solution;
File vendor/reviewscouk/reviews/Observer/SendOrderDetails.php needs a rewrite so that a lookup is done from the product_id to retrieve the original SKU for the simple product so that the review invitation data which match/link to data in the feed.

Verified Buyer text missing in Static Content Widget

Could you add in the 'verified buyer: yes' text that is in the javascript widget into the static content widget? We've switched to the static content widget to improve SEO, but the 'verified buyer' text is crucial too:

richSnippet SKU with double-quote

Issue description

When configurable product SKUs have double-quotes in them (although I realize double-quotes in SKUs are not ideal, that is what exists), then the richSnippet JS breaks. See screenshot below.

I think the most ideal solution would be to replace the double-quotes in the SKU data attribute in \Reviewscouk\Reviews\Block\Richsnippet::getRichSnippet with "template literals" (backticks). This is already done with the description.

If this is an acceptable solution, I can create a PR. Thanks.

Steps to reproduce the issue

1. Create a configurable product with a SKU that contains a double-quote.
2. Navigate to that PDP.

What's the expected result?

• No JS errors

What's the actual result?

• The following JS error shows in the console:

Uncaught SyntaxError: Unexpected token ';' (at sample-url:2347:323)

Additional details / screenshot

• 
• 

Change rich snippet info for review's Author.

Hi there.
I need to change rich snippet info for Author. I mean the google Rich results test shows that Author's type is not mentioned. Could You please say how to modify it?

Exception #0 (Exception): Notice: Trying to access array offset on value of type null in /vendor/reviewscouk/reviews/Block/Richsnippet.php on line 51

Hello,

I have this error when accessing the checkout - Exception #0 (Exception): Notice: Trying to access array offset on value of type null in /Users/steven/Sites/suregreen/vendor/reviewscouk/reviews/Block/Richsnippet.php on line 51

The line in question is below:

'availability' => $this->availability($current_product->getData('quantity_and_stock_status')['is_in_stock']),

We are using a onestepcheckout from Mageplaza - mageplaza/magento-2-one-step-checkout-extension

The version of the extension is "0.0.40".

Is this a conflict between your extension and the mageplaza one?

I have come across this issue when upgrading to magento 2.4.3.

Thank you for your help.

Feature request: option of cancelling a scheduled review request email when an order is refunded

Sometimes customers that return an item will still receive an email notification for a review. I'm wondering if we can hook into the credit memo/refund event in Magento and cancel the scheduled email in those situations.

a) is this on the roadmap, and if not
b) would you accept a pull request that adds this functionality (behind a toggle flag in admin store config)?

Non Composer Installation

I am not using composer to add extensions - do you have the full files I can download so I can upload to my site via FTP and run magento upgrade?
Or point me to the right directopn please
Thanks

Pull through average review rating for simple and configurable products

Hello,

This isn't an issue - more a feature request. Is it possible to pull through the products average review summary (similar to Magento's default reviews) under the product title on the product page (& on the category page ideally) using your module? Currently we're using the reviews.co.uk api to get this information, but having issues with configurable products, so if it was included in your module that would be great!

Thanks

Missing ACL - no ability to give user roles access to configuration

Your module does not include the ACL configuration so it's not possible to grant access to any users who do not have full administration rights.

Some helpful resources:

https://magento2tutorial.magestore.com/how-to-declare-acl-for-the-configuration-page-of-custom-module-in-magento-2/
https://devdocs.magento.com/guides/v2.4/ext-best-practices/tutorials/create-access-control-list-rule.html

Deprecated Functionality: strpos(): Passing null to parameter #1

reviewscouk/reviews/Block/Product/Reviewwidget.php line 112 if colour is NULL in the admin this line fails. replacing the line with sorts this.

if (strpos($colour ?? '', '#') === false) {

Widgets are HTML escaped

Hey,

Widgets, which include HTML, are being escaped and thus don't work.

Thanks,
Warren

PHP Compat Issue When Using v2.4.4-p2

When using Magento Open Source v2.4.4-p2 we are getting the following error messages on product pages;

Exception #0 (Exception): Deprecated Functionality: strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in /var/www/html/vendor/reviewscouk/reviews/Block/Product/Reviewwidget.php on line 112