Although this module does have an existing README, it lacks some tips on functionality and configuration.

According to Drupal standards, modules should include a useful README.file.
https://www.drupal.org/docs/develop/documenting-your-project/module-docu....

README to follow. Thanks!

CommentFileSizeAuthor
#5 interdiff_2-5.txt3.22 KBdhruveshdtripathi
#5 readme_needed-2849557-5.patch2.93 KBdhruveshdtripathi
#2 i18nviews-add-readme-2849557-2.patch2.85 KBvolkswagenchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
Assigned: volkswagenchick » Unassigned
Issue summary: View changes
Status: Needs work » Needs review
FileSize
i18nviews-add-readme-2849557-2.patch2.85 KB

Uploaded a patch with a README that contains some useful tips for configuration and functionality. THANKS!

loopduplicate’s picture

loopduplicate
she/her
Location Burning Token, California
CreditAttribution: loopduplicate at Hook 42 commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Looks good to me :)

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Needs work

Needs a minor tweak to wrap at 80 characters.

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs work » Needs review
FileSize
readme_needed-2849557-5.patch2.93 KB
interdiff_2-5.txt3.22 KB

Three minor changes made:

  • Numbered lists indented 4 spaces.
  • Bulleted lists indented 1 space.
  • Text manually word-wrapped within around 80 cols.
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
Status: Needs review » Reviewed & tested by the community

Thanks for reviewing this patch! When I initially wrote this patch I was not fluent in documentation standards, thanks for your patience.

Patch #5 looks good, marked RTBC.