- 🇺🇸United States Delty
It might be worth checking out how Simplenews handles unsubscribe requests for anonymous users since the use case is almost identical.
Thanks,
-=Delty
- 🇺🇸United States jcmartinez Raleigh, NC, USA
This module probably worked for a very specific use case, but it isn't useful for a general ecommerce site, unless this problem is solved.
It was very close to what I needed.
- @jcmartinez opened merge request.
- 🇺🇸United States jcmartinez Raleigh, NC, USA
I just created an Issue Fork with a fix that looks into permissions to allow (or not) anonymous users to create notifications. Off course, this will depend on the permission to be granted by you to the anonymous role.
It works for me on the Dev version of this module.
You can use composer to apply a patch from the plain diff link under the Issue Fork.
Remeber that if you allow anonymous users to subscribe, then you will be in charge of unsubscribing them because they can't do it by themselves.
- 🇨🇭Switzerland Lukas von Blarer
There is another issue regarding this: ✨ Proposed new features patch - allow anonymous, unsubscribe if duplicate Closed: duplicate
- 🇺🇸United States jcmartinez Raleigh, NC, USA
This patch is a remake of my previous patch, to be used after applying two other patches that will bring this module to compatibility with Drupal 10.
Add the following three patches to the patches section of your composer.json file to solve the current issue and to add compatibility to Drupal 10:
"patches": { "drupal/commerce_stock_notifications": { "Drupal 10 compatibility": "https://git.drupalcode.org/project/commerce_stock_notifications/-/merge_requests/11.diff", "Drupal 10 compatibility Continued": "https://git.drupalcode.org/project/commerce_stock_notifications/-/merge_requests/14.diff", "Allow anonymous create notifications": "https://www.drupal.org/files/issues/2023-11-11/3215770-D10-anonymous-notifications.patch" } }
When the patches that add compatibility to this module are merged, we should be able to use this patch alone to solve the current issue under Drupal 10.
- Status changed to Needs work
8 months ago 1:44pm 28 April 2024 - 🇸🇰Slovakia poker10
Thanks for working on this. I have added some comments to the MR.
- 🇸🇰Slovakia poker10
I have moved this to a bug, because we have a permission
create commerce stock notification
, but it is not used at all. -
poker10 →
committed 20926f6e on 8.x-1.x authored by
jcmartinez →
Issue #3215770: anonymous cannot create notifications although /...
-
poker10 →
committed 20926f6e on 8.x-1.x authored by
jcmartinez →
- Status changed to Fixed
7 months ago 8:51pm 31 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.