According to Drupal standards, modules should include a useful README.file.
While this project has a readme, it is lacking useful configurations and links.

https://www.drupal.org/docs/develop/documenting-your-project/module-docu...
Patch to follow. Thanks!

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new3.19 KB

I have uploaded a patch that aligns with Drupal documentation standards and include useful links and configurations.
Thanks for your work on this project!!

legaudinier’s picture

Status: Needs review » Reviewed & tested by the community

Patch applied using simplytest.me

Readme file is more useful. Grammar and spelling look good. Links are correct.

Marking RTBC.

snater’s picture

Thanks. Would it be OK to have the configuration section just point to https://www.drupal.org/docs/8/modules/insert/configuration in order to avoid having to maintain documentation in two places?

volkswagenchick’s picture

StatusFileSize
new3.34 KB
new467 bytes

I do see your point. I have a link to the external documentation in the Introduction section.

The only problem I see to ONLY linking to external documentation is that you have to be on-line to access the information. There are several use cases where this might not work: working locally with no internet access, can only view whitelisted urls, etc.

Could we add something like

"For the most up to date configurations, please visit the external documentation pages at:
https://www.drupal.org/docs/8/modules/insert/configuration"

Patch uploaded with an interdiff for easy review.

I appreciate your feedback. If you want it some other way, let me know. :)

snater’s picture

OK, not having any Internet access at all may be a use-case, but it is a quite narrow one as the module has to be downloaded somewhere to get started. While I try to document as good as possible, I neither want to have the obligation to maintain documentation in more than one place, nor that the user might accidentally be reading outdated documentation. If drupal.org had a system like GitHub where the content of the readme is integrated into the software's presentation, I would very well be fine with putting all the documentation into the readme instead of somewhere else. Under the given circumstances, in my opinion, it is a good idea to have the readme file be some kind of index to all necessary information (which is what your change does), but I would prefer to not copy specific documentation into that file.

That said, I do not have a problem uploading your patch. I just do not want to be impolite by silently making a major change to your change in a subsequent commit, which is what I would do for the current patch set. But if you are fine with that, I will just go ahead and upload your change set.

volkswagenchick’s picture

Thanks for asking. I see your point of view.

Go ahead and make the changes you feel necessary. No offense will be taken :)

snater’s picture

Status: Reviewed & tested by the community » Fixed

OK, thanks a lot. :)

volkswagenchick’s picture

Thankyou for all the work you do!!
I am encouraging team mates to help out on Drupal.org, could you go back and give legaudinier a credit for his RTBC?? She took the time to test and make sure the configs, link, etc were correct.
The box next to her name just gets ticked and she 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

snater’s picture

(Sorry, I am still having trouble with those credit mechanics.) I am not able to amend the commit, but I did tick her name in the credit section when I set the issue to fixed (as she, of course, deserves credit). According to her user page, she received credit on that issue.

Status: Fixed » Closed (fixed)

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