Skip to content

Commit

Permalink
Dev: Remove email from test surveys
Browse files Browse the repository at this point in the history
  • Loading branch information
@olleharstedt
olleharstedt committed Jun 27, 2017
1 parent 0e74160 commit c7ceddb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions tests/data/surveys/limesurvey_survey_454287.lss
View file
Expand Up @@ -230,7 +230,7 @@
<row>
<sid><![CDATA[454287]]></sid>
<admin><![CDATA[Paulo Oliveira]]></admin>
<adminemail><![CDATA[psoliveira@med.up.pt]]></adminemail>
<adminemail><![CDATA[tmp@tmp.com]]></adminemail>
<anonymized><![CDATA[N]]></anonymized>
<faxto/>
<format><![CDATA[G]]></format>
Expand All @@ -257,7 +257,7 @@
<assessments><![CDATA[N]]></assessments>
<usecaptcha><![CDATA[N]]></usecaptcha>
<usetokens><![CDATA[N]]></usetokens>
<bounce_email><![CDATA[psoliveira@med.up.pt]]></bounce_email>
<bounce_email><![CDATA[tmp@tmp.com]]></bounce_email>
<emailresponseto/>
<emailnotificationto/>
<tokenlength><![CDATA[15]]></tokenlength>
Expand Down
4 changes: 2 additions & 2 deletions tests/data/surveys/limesurvey_survey_917744.lss
View file
Expand Up @@ -175,7 +175,7 @@
<row>
<sid><![CDATA[917744]]></sid>
<admin><![CDATA[Paulo Oliveira]]></admin>
<adminemail><![CDATA[psoliveira@med.up.pt]]></adminemail>
<adminemail><![CDATA[tmp@tmp.com]]></adminemail>
<anonymized><![CDATA[N]]></anonymized>
<faxto/>
<format><![CDATA[G]]></format>
Expand All @@ -202,7 +202,7 @@
<assessments><![CDATA[N]]></assessments>
<usecaptcha><![CDATA[N]]></usecaptcha>
<usetokens><![CDATA[N]]></usetokens>
<bounce_email><![CDATA[psoliveira@med.up.pt]]></bounce_email>
<bounce_email><![CDATA[tmp@tmp.com]]></bounce_email>
<attributedescriptions><![CDATA[{"attribute_1":{"description":"sexo","mandatory":"N","show_register":"N","cpdbmap":""}}]]></attributedescriptions>
<emailresponseto/>
<emailnotificationto/>
Expand Down

7 comments on commit c7ceddb

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But tmp/./com exist :). limsurvey.org or example.com seems better :) (i do it tomorrow if i remind)

@olleharstedt
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. Poor psoliveira can expect some fresh spam...

@olleharstedt
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOT limesurvey.org - this is a test survey, it's not ment to be read, ever.

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you check other lss ? I do a check and fix it with example@example.com this morning :).

@olleharstedt
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, didn't check anything else.

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done for lss files : only one email.com still here. I remove the name too :)

@olleharstedt
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, great!

Please sign in to comment.