Expose wipejob action via TestSwarm API. #204

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

jayarjo commented Jul 2, 2012

Another try :)

@ghost ghost assigned Krinkle Jul 2, 2012

Member

Krinkle commented Jul 2, 2012

Thanks! Looks good on first sight. Will test later tonight.

+ $authToken
+ ));
+
+ if ( $authUserName !== $jobOwner ) {
@Krinkle

Krinkle Jul 4, 2012

Member

Fo your information:

Notice: Undefined variable: authUserName in /Users/krinkle/Development/github/jquery/testswarm/inc/actions/WipejobAction.php on line 74

("N" instead of "n", causing variable mismatch, so this this never worked).

Don't worry about amending right now, as I'm including this in #209 (keeping you as committer in the metadata).

@jayarjo

jayarjo Jul 5, 2012

Contributor

I really should get used to writing a code with notices turned on. Thanks.

@Krinkle Krinkle closed this in 2073672 Jul 5, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment