Skip to content

Conversation

@deobald
Copy link
Contributor

@deobald deobald commented Mar 31, 2012

  1. Fixed bug: install a new jukebox2, start it up. Everything's cool. Now create a new user. Restart the server. It goes into infinite loop, looking for the new user's music. We fixed the infinite loop.

  2. We also create an empty dir for new users on signup now.

@dan-manges
Copy link
Collaborator

I just merged this into thehammer/jukebox2/master and I'm getting this spec failure:

Failures:

  1) library random-song returns the default song if no user songs exist
     Expected truthy but was: <false>
     jukebox2/spec/jukebox_web/models/library_spec.clj:61

Does lein spec pass for you if merging this branch into thehammer/master?

@deobald
Copy link
Contributor Author

deobald commented Apr 18, 2012

Hey Dan, this failure is similar to the previous one on the bulk_queue branch. Rather than repeat the fix in this branch and potentially cause conflicts for you, I'll just hold off until bulk_queue is merged.

Sorry about the breaks! Sloppy Saturday coding.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants