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

SHS-5986: Uninstall Default Paragraphs module #1710

Closed

Conversation

codechefmarc
Copy link
Collaborator

@codechefmarc codechefmarc commented Jan 7, 2025

READY FOR REVIEW

Summary

Uninstalls hs_paragraphs_between and paragraphs_browser as they have been replaced with functionality from other modules. Composer changes to be removed from codebase in a subsequent codebase after release.

Need Review By (Date)

2025-01-08

Urgency

low

Steps to Test

  1. Login to the site as a developer
  2. Visit the modules page at /admin/modules
  3. Verify that H&S Paragraphs Between and Paragraph Browser are both uninstalled
  4. Verify that, when adding Flexible and Private pages, there is still an "add-in-between" option for paragraphs and that the functionality still works as expected.

PR Checklist


@ahughes3 ahughes3 temporarily deployed to Tugboat January 7, 2025 23:53 Destroyed
@codechefmarc codechefmarc self-assigned this Jan 8, 2025
@codechefmarc codechefmarc marked this pull request as ready for review January 8, 2025 00:10
Base automatically changed from 11.6.1-release to develop January 8, 2025 16:39
Copy link
Collaborator

@callinmullaney callinmullaney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@codechefmarc PR looks good on my end and the requested modules have been uninstalled:
Screenshot 2025-01-08 at 1 53 26 PM

@callinmullaney
Copy link
Collaborator

@ahughes3 This PR is looking good to me. This should cover the second-to-last step in the process of removing these modules from the code base. Ready for you to confirm they are uninstalled.

@ahughes3 ahughes3 self-requested a review January 9, 2025 21:20
Copy link
Collaborator

@ahughes3 ahughes3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ahughes3 ahughes3 assigned pookmish and unassigned ahughes3 Jan 9, 2025
@ahughes3 ahughes3 requested review from pookmish and sethprime January 9, 2025 21:21
Copy link
Collaborator

@sethprime sethprime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified that H&S Paragraphs Between, and Paragraphs Browser modules are uninstalled.
Verified that you can add paragraphs between on flexible and private pages.

Copy link
Member

@pookmish pookmish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because this will conflict with #1698, let's merge the two PRs together and combine the update hook into one.

Comment on lines +1193 to +1194
$config_factory = \Drupal::configFactory();
$config_factory->getEditable('paragraphs_browser.paragraphs_browser_type.content')->delete();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't necessary.

/**
* Uninstall hs_paragraphs_between and paragraphs_browser.
*/
function su_humsci_profile_update_9721() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will conflict with the other PR to uninstall honeypot.

@codechefmarc
Copy link
Collaborator Author

@pookmish - thank you for the comments, I've combined these two PRs into one, here: #1713

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants