Success

User tests: Successful: Unsuccessful:

avatar dbhurley
dbhurley
16 Nov 2013
avatar dbhurley dbhurley - open - 16 Nov 2013
avatar dbhurley dbhurley - open - 16 Nov 2013
avatar elkuku
elkuku - comment - 16 Nov 2013

I'd like something to identify the action of a machine (a script).

avatar b2z
b2z - comment - 16 Nov 2013

@elkuku why you need that?

avatar elkuku
elkuku - comment - 16 Nov 2013

Hmm, maybe it's just me but I' think that during testing we should have some "mark" to identify that this comment has not been directly submitted to GitHub by a human being...

I am not opposed to remove it in a final version (hence the @todo removeMe note in the code ;) )

If you, however, trust the machines - go push that merge button ;)

avatar mbabker
mbabker - comment - 16 Nov 2013

I agree with @elkuku on this - It does help to see what came from the app and troubleshoot if need be but it shouldn't be in the final version.

avatar dbhurley
dbhurley - comment - 16 Nov 2013

No worries, then we need to sort out why it doesn't get parsed out on the
follow-up comments.

Thanks,
David Hurley
On Nov 16, 2013 11:57 AM, "Michael Babker" notifications@github.com wrote:

I agree with @elkuku https://github.com/elkuku on this - It does help
to see what came from the app and troubleshoot if need be but it shouldn't
be in the final version.


Reply to this email directly or view it on GitHub#218 (comment)
.

avatar elkuku
elkuku - comment - 16 Nov 2013

IIRC the \n characters are somehow stripped (that's why I included a <br /> here) and a blank line is required to separate the quoted text.
Feel free to play around here ;)

avatar dbhurley
dbhurley - comment - 16 Nov 2013

Yep, I was looking at it yesterday. I'll see if I can tell why it's not
sorted out.

Thanks,
David Hurley
On Nov 16, 2013 12:27 PM, "Nikolai Plath" notifications@github.com wrote:

IIRC the \n characters are somehow stripped (that's why I included a
/> here) and a blank line is required to separate the quoted text.
Feel free to play around here ;)


Reply to this email directly or view it on GitHub#218 (comment)
.

avatar dbhurley dbhurley - close - 27 Nov 2013

Add a Comment

Login with GitHub to post a comment