While evaluating this module, I noticed that the README is missing a dependency that is outside of Drupal core.

Patch to follow, thanks!

CommentFileSizeAuthor
#2 paragraphs_wizard_d8-fix-readme-2949161-2.patch2.49 KBvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review
StatusFileSize
new paragraphs_wizard_d8-fix-readme-2949161-2.patch2.49 KB

I have uploaded a patch that adds the missing dependency to the README. I also took the time to re-format the README to better align with Drupal.org documentation standards, thanks!
README Template
https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

dani3lr0se’s picture

dani3lr0se commented
Status: Needs review » Reviewed & tested by the community

Patch applies cleanly. README looks good. Looks like the issue was addressed as the dependencies were added to the file. I can't think of any other suggestions. Looks good to me. Thanks for the patch.

rafuel92’s picture

rafuel92 commented
Status: Reviewed & tested by the community » Fixed
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

Thankyou for all the work you do!!
I am encouraging team mates to help out on Drupal.org, could you go back and give daniel_rose a credit for his RTBC?? The box next to his name just gets ticked and he gets a credit! I see it as a way of incentive to help get involved it the community! It takes a village.

Cheers!

AmyJune - volkswagenchick

rafuel92’s picture

rafuel92 commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.