Would just like to update the README.txt to show the new help and support. ;)

Patch to follow.

CommentFileSizeAuthor
#6 update-readmetxt-2931794-2.patch1.42 KBdani3lr0se
#2 update-readme-txt-2931794-2.patch1.13 KBdani3lr0se
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

daniel_rose created an issue. See original summary.

dani3lr0se’s picture

dani3lr0se CreditAttribution: dani3lr0se at Hook 42 commented
Status: Active » Needs review
FileSize
update-readme-txt-2931794-2.patch1.13 KB

Here is a patch that adds some new maintainers and another support organization. :)

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

Looks good. Applied the patch using SImplytest.me and it applied clean. This is readme documentation, so there was no UI tesing to be done. Marking as RTBC

andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented

@daniel_rose, you missed me in module maintainers :)

dani3lr0se’s picture

dani3lr0se CreditAttribution: dani3lr0se at Hook 42 commented

My apologies. I can fix that. :)

dani3lr0se’s picture

dani3lr0se CreditAttribution: dani3lr0se at Hook 42 commented
Status: Reviewed & tested by the community » Needs review
FileSize
update-readmetxt-2931794-2.patch1.42 KB

Here is a new patch that adds @andrey.troeglazov and also corrects a mistake I made earlier by not changing the reference numbers in the second paragraph.

dani3lr0se’s picture

dani3lr0se CreditAttribution: dani3lr0se at Hook 42 commented

Someday I will remember to change the comment number in patches if I update one. I think I always forget that. Would also help if I did it right the first time. :(

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

The patch in Comment #6 is misnamed, but i tested it using simplytest.me and patch applies. No UI testing as this documentation.
Reference numbers look good and coincide to the appropriate entities. Comment #4 was addressed. :)

thanks

  • daniel_rose authored cf74da2 on 7.x-6.x 
    Issue #2931794 by daniel_rose, volkswagenchick, andrey.troeglazov:...
andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented
Version: 7.x-5.x-dev » 7.x-6.x-dev
Status: Reviewed & tested by the community » Fixed

Thank you for patch and testing. I have also added my company to the list and commited :)
If you can work on other issues it will be good, I will be glad to help you and review it, cause I'm hard-working now on Nivo slider module.
Thanks!

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented

Thankyou andrey.troeglazov!!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

Thanks @andrey.troeglazov. BTW please keep the CHANGELOG.txt files updated too.

Status: Fixed » Closed (fixed)

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