AS per a slack conversation:

Greg Knaddison 7:53 AM
on https://simplytest.me/qa it says What is the login data for a sandbox site? but if you happen to be testing the https://www.drupal.org/project/email_registration/ module then you need to know the email address
I was able to figure out the email address is admin@example.com but it seems nice to document that somewhere.

Please add the email address to the readme and supporting docs.

CommentFileSizeAuthor
#12 simplytest-readme-3106649-12.patch497 bytesvolkswagenchick
#4 simplytest-readme-3106649-4.patch250 bytesvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

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

I have assigned this to myself to save as a novice issue while on-boarding new contributors. Thanks.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Parent issue: » #3105930: [meta] Internal documentation guide
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Version: 8.x-4.x-dev » 7.x-1.x-dev
Assigned: volkswagenchick » Unassigned
Issue tags: +Novice, +DCCO2020
StatusFileSize
new simplytest-readme-3106649-4.patch250 bytes

I am "at" DrupalCamp Colorado onboarding a new contributor. We reviewed the patching process and I am uploading a patch that adds the email credential to the readme.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Active » Needs review
wperels’s picture

wperels commented
Assigned: Unassigned » wperels
Status: Needs review » Reviewed & tested by the community

We are "at" DrupalCamp Colorado. volkswagenchick showed me how to create a patch that adds the email credential to the readme. I have reviewed the patch and it looks good.

Webbeh’s picture

Webbeh
he/him/his
Location Georgia, USA
commented
Assigned: wperels » Unassigned

volkswagenchick credited greggles.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

You are the best, volkswagenchick. Thank you :)

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: Unassigned » volkswagenchick
Status: Reviewed & tested by the community » Needs work

Moving this to needs work. There are actually 2 places in the read me where the email creds should be added.

I am assigning this to myself and will be using this issue as an example in contribution WS tomorrow.

Please save this issue for that demo. Thanks.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review
Issue tags: +Documentation
StatusFileSize
new simplytest-readme-3106649-12.patch497 bytes

I am uploading a patch that adds the email creds in both relevant locations.

Marking needs review. I am unassigning myself.

Webbeh’s picture

Webbeh
he/him/his
Location Georgia, USA
commented
Status: Needs review » Reviewed & tested by the community

+1 RTBC, this looks good to me.

beautifulmind’s picture

beautifulmind
he/him
Primary language Gujarati
Location Dubai
commented

+1 RTBC.
Please proceed to close this issue. This has long been overdue.

Regards.