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

Updating drupal to 10.3.1 #632

Merged
merged 4 commits into from
Aug 1, 2024
Merged

Conversation

ameshkin
Copy link
Contributor

The rules module was removed to get the site working with 10.3.1. However, this module is used to send an email when a new user signs up to the site. I'm creating this draft PR for now to easily compare to develop. Once there is a patch or update to the rules module, I'll update the branch and all commits will be squashed into one.

FILE FOR SENDING EMAIL:
https://github.com/usdoj/foia-api/blob/develop/config/default/rules.reaction.new_user_notification.yml

ISSUE:
https://www.drupal.org/project/rules/issues/3454056

@ameshkin
Copy link
Contributor Author

Added config_split module which fixed github test error. Still not ready to merge without a fix for drupal/rules.

…htning core.

FOIA-367: Pushing up code without rules module which is used to send new user account email, do not merge.

FOIA-467: Adding config split module due to error when running tests.

FOIA-467: Adding lightning media core that may fix github action test.

[FOIA22-168] Content changes to support new CSS

FOIA-467: Removing drupal/rules completely.
@ameshkin ameshkin marked this pull request as ready for review July 22, 2024 14:14
@ameshkin ameshkin marked this pull request as draft July 22, 2024 15:53
@ameshkin ameshkin marked this pull request as ready for review July 24, 2024 15:56
@ameshkin ameshkin marked this pull request as draft July 24, 2024 15:57
@ameshkin
Copy link
Contributor Author

Added rules back to composer.json file.

@ameshkin ameshkin marked this pull request as ready for review July 24, 2024 16:07
@brockfanning brockfanning merged commit 09c28c5 into usdoj:develop Aug 1, 2024
1 check passed
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