πŸ‡¬πŸ‡§United Kingdom @karengrey

Account created on 8 October 2009, about 15 years ago
  • Senior Drupal Developer at CentarroΒ  …
#

Merge Requests

More

Recent comments

πŸ‡¬πŸ‡§United Kingdom karengrey

Thank you generalredneck, this is exactly what I needed!

πŸ‡¬πŸ‡§United Kingdom karengrey

@hanoii I think theres a wider problem here. Drupal 9.5.11 requires symfony/event-dispatcher-contracts v1.1.13, but a dependency of this module is solarium/solarium which now requires symfony/event-dispatcher-contracts ^2.0 || ^3.0

Im not sure what the solution is here, as im currently trying to do a Drupal 10 upgrade from 9.5.11, but need to update the modules first

πŸ‡¬πŸ‡§United Kingdom karengrey

karengrey β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom karengrey

Added functionality to allow custom CSS to be applied to the iFrame in the configuration form. Default CSS will be applied if not enabled.

πŸ‡¬πŸ‡§United Kingdom karengrey

I've applied the proposed solution and seems to work well. Have ran through as many scenario's I could think of to stop any errors/bugs. MR created and pushed up for review.

πŸ‡¬πŸ‡§United Kingdom karengrey

karengrey β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom karengrey

Once the Commerce Core issue has been committed, this MR will be unblocked and will add a new EmailEvent for registration during checkout.

πŸ‡¬πŸ‡§United Kingdom karengrey

Created issue for Commerce Core to create the new Event for registration during checkout: https://www.drupal.org/project/commerce/issues/3388613 πŸ“Œ Create Event for registration during checkout Active

πŸ‡¬πŸ‡§United Kingdom karengrey

I have updated the patch based on Ryan's suggestion to check if the site is setup for HTML email. I have added a check for symfony_mailer, as this is the suggested module to use for HTML emails (swiftmailer has been unsupported since November 2021)
I have also included all text formats, so the user has a choice.

I saw no other config available to check if a site is setup to use HTML email, as Core MailManager only returns the default `php_mail` interface.

πŸ‡¬πŸ‡§United Kingdom karengrey

karengrey β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom karengrey

Failed tests using PHP 8.1 are unrelated to this project and issue, so should be ignored.

πŸ‡¬πŸ‡§United Kingdom karengrey

Additional functionality added to create a configurable option to allow anonymous Wishlists when shared by email, to be duplicated and Wishlist items added to the new Wishlist. The duplicated Wishlist is the one that is emailed to the recipient.

πŸ‡¬πŸ‡§United Kingdom karengrey

karengrey β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom karengrey

karengrey β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom karengrey

I have marked this issue as a duplicate of https://www.drupal.org/project/commerce_wishlist/issues/3170216 ✨ Provide option to only add item to wishlist once. Needs review

πŸ‡¬πŸ‡§United Kingdom karengrey

Rather than hiding the sharing functionality from anonymous users, we should give the option for how to handle this.

Anonymous sharing of a list should be possible. You should be able to share a direct link to a Wishlist and have an email sent off that contains the link.

Anonymous sharing should be able to be disabled. i.e. admin should be able to turn off anonymous sharing and require that someone be logged in in order to share via email

πŸ‡¬πŸ‡§United Kingdom karengrey

karengrey β†’ made their first commit to this issue’s fork.

Production build 0.71.5 2024