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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

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

volkswagenchick’s picture

volkswagenchick’s picture

Version: 8.x-4.x-dev » 7.x-1.x-dev
Assigned: volkswagenchick » Unassigned
Issue tags: +Novice, +DCCO2020
FileSize
250 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

Status: Active » Needs review
wperels’s picture

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

Assigned: wperels » Unassigned

volkswagenchick’s picture

greggles’s picture

You are the best, volkswagenchick. Thank you :)

volkswagenchick’s picture

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

Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review
Issue tags: +Documentation
FileSize
497 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

Status: Needs review » Reviewed & tested by the community

+1 RTBC, this looks good to me.

beautifulmind’s picture

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

Regards.