search

TheLastCrusade / SoundStream

1 stars 6 forks source link

Adds test workflows to github in markdown #259

Closed taywrobel closed 11 years ago

taywrobel commented 11 years ago

Adds a markdown version of the updated test script and workflows to github, so that it can be maintained by the entire team and reviewd by beta participants.

Adds script to git - fixes #251 Removes incorrect disband check in test 35 - fixes #202 Includes check for toast when removing last song - fixes #246 Adds reconnect to test 24 - fixes #247

theJenix commented 11 years ago

The tests that result in an empty playlist must indicate which phone (guest or host) will display the "Playlist is empty" message.

reidbaker commented 11 years ago

I would love to get rid of all the question marks. That should not hold off getting this pulled in but if it doesnt get fixed please make a ticket.

dgreenhalgh commented 11 years ago

@twrobel3,could we get the changes made and have this pulled in by this Sunday's meeting? I'd love to start looking forward at what's next.

taywrobel commented 11 years ago

Yeah, it's been a crazy week, but I can take care of it tomorrow.

On Fri, Aug 9, 2013 at 8:45 AM, David Greenhalgh notifications@github.comwrote:

@twrobel3 https://github.com/twrobel3,could we get the changes made and have this pulled in by this Sunday's meeting? I'd love to start looking forward at what's next.

— Reply to this email directly or view it on GitHubhttps://github.com/TheLastCrusade/SoundStream/pull/259#issuecomment-22403100 .

Taylor Wrobel

dgreenhalgh commented 11 years ago

No worries, thanks man On Aug 9, 2013 3:44 PM, "Taylor Wrobel" notifications@github.com wrote:

Yeah, it's been a crazy week, but I can take care of it tomorrow.

On Fri, Aug 9, 2013 at 8:45 AM, David Greenhalgh notifications@github.comwrote:

@twrobel3 https://github.com/twrobel3,could we get the changes made and have this pulled in by this Sunday's meeting? I'd love to start looking forward at what's next.

— Reply to this email directly or view it on GitHub< https://github.com/TheLastCrusade/SoundStream/pull/259#issuecomment-22403100>

.

Taylor Wrobel

  • www.linkedin.com/in/taylorwrobel/

— Reply to this email directly or view it on GitHubhttps://github.com/TheLastCrusade/SoundStream/pull/259#issuecomment-22418879 .

taywrobel commented 11 years ago

@theJenix could you take a look and make sure that all the changes were addressed? If so this should be good to push in.

theJenix commented 11 years ago

:+1: Looks good to me. :pushpin: