Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Drupal 10.3 #1224

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Update to Drupal 10.3 #1224

wants to merge 7 commits into from

Conversation

a-pasquale
Copy link
Contributor

Describe context / purpose for this PR

Link to MultiDev instance

http://md-103-accessmatch.pantheonsite.io

Checklist for PR author

  • I have checked that the PR is ready to be merged
  • I have reviewed the DIFF and checked that the changes are as expected
  • I have assigned myself or someone else to review the PR

@a-pasquale a-pasquale changed the title Update to Drupal 10.3.1 Update to Drupal 10.3 Sep 13, 2024
@a-pasquale
Copy link
Contributor Author

@protitude The rules sending email notifications are triggering this error:
TypeError: Drupal\typed_data\PlaceholderResolver::scan(): Argument #1 ($text) must be of type string, null given, called in /app/web/modules/contrib/rules/src/Plugin/RulesDataProcessor/TokenProcessor.php on line 69 in Drupal\typed_data\PlaceholderResolver->scan() (line 178 of modules/contrib/typed_data/src/PlaceholderResolver.php).

e.g. /admin/config/workflow/rules/reactions/edit/new_user?destination=/admin/config/workflow/rules

Can you take a look and see if you can figure out tokens that don't cause the issue some other fix? If not, we could recreate the rules in code and maybe just get rid of the rules module?

@a-pasquale a-pasquale marked this pull request as draft October 1, 2024 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants