Closed (fixed)
Project:
Delete all
Version:
8.x-1.x-dev
Component:
Documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Jul 2019 at 04:18 UTC
Updated:
18 Dec 2021 at 09:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
joey-santiago commentedhere's a patch, was indeed quite easy to use this module :)
Comment #3
WebbehComment #4
Webbeh#2 needed work.
* Standardized header spacing and formatting.
* Standardized requirements, installation copy.
For review.
Comment #5
karimb commentedNice job of @Webbeh!
I still found some small issues and fix them.
* Change ?<80>? (small arrow) for > (normal arrow)
* Add specific navigation and links for Drupal 8
Need review.
Comment #6
WebbehShould we remove the D7 documentation from the README (for the 8.x-1.x branch), and just change USAGE (Drupal 8) to USAGE?
Comment #7
mrinalini9 commentedComment #8
mrinalini9 commentedUpdated patch as per the changes mentioned in #6, please review.
Comment #9
paulocsThe patch #8 looks good!
I think it could have the drupal 8 drush commands on the module page.
At the moment we can only see the commands from drupal 7.
Should I open an issue for it?
Thanks :)
Comment #10
WebbehThat'd be my recommendation. Create a new issue, and reference this one in the OP or related issue.
Comment #11
karimb commentedNice job @mrinalini9, @paulocs and @Webbeh
Small typo fix, "this actions" to "these actions".
-> Needs review.
Comment #12
manav commentedThe above patch #11 is working fine for me but there is some typo mistake in it.
I have correct it like:
Line no 21: modules that needs to be cleaned
changes to: modules that need to be cleaned
Line no 34: outside of Drupal core.
changes to: outside of the Drupal core.
--> Needs review
Comment #13
manav commentedComment #14
WebbehSome small nits to #11, mainly focused on active voice and adding in the menu breadcrumbs alongside the URL slug. For review.
Comment #15
karimb commentedThe patch #14 looks good!
Comment #16
gthing commentedCan this be also added to 2.x?
Comment #17
gthing commentedSome documentation says you can delete all nodes of a content type with drush delete-all whatevercontenttype. What I'm seeing here doesn't look like it refers to managing specific content types and when executing the command it only asks generically "do you want to delete the node?" I'm not quiteee sure what it's doing...
Comment #19
dipakmdhrm commented