Skip to content

Conversation

@meltar
Copy link
Owner

@meltar meltar commented Apr 29, 2015

Replaces champions with ✨ emoji ✨ 🎸

In order to make this work, production users will need to be migrated to appropriate or random emoji or reset. New users will be automatically assigned a random emoji instead something from the former champion list.

To do:

  • Fix specs
  • Add tests for two new helper methods
  • Write rake task (or prepare to migrate names manually)

To discuss:

  • Get rid of champions and make the champion name an attribute of the user?
  • Review respond_to and respond_with blocks (and Responder gem)

Note: I should have gotten the tests running but I kept finding more things that were broken, and it's time to sleep. 🌃

Edit: I guess the next step is making Travis happy. 😳

@meltar
Copy link
Owner Author

meltar commented Apr 29, 2015

When you get a chance - @KyFaSt Thoughts? Should I work on the to do list?

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