Skip to content

Conversation

pvcarrera
Copy link
Contributor

No description provided.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is a small typo. anohter -> another

@despo
Copy link
Member

despo commented Nov 5, 2013

@pvcarrera thanks so much for this 👍 . Do you have sometime to look at the feedback? If not I can refactor a bit later :)

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 635ae29 on pvcarrera:session-host into e0ef26a on codebar:master.

@pvcarrera
Copy link
Contributor Author

@despo I'm very happy to help. I'm learning a lot about rails following your code and thinking about how to solve the task. If you are not in a hurry I'll do it tomorrow it's quite late to me :-)

@despo
Copy link
Member

despo commented Nov 5, 2013

@pvcarrera no hurry! Thank you 😄 Next set of invitations wont be going out until Thursday at least, so we have time!

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling fe68e41 on pvcarrera:session-host into e0ef26a on codebar:master.

@despo
Copy link
Member

despo commented Nov 6, 2013

👍 This is perfect @pvcarrera thank you! Sorry for taking a while to have a look. Tuesday nights are tutorial nights.. Hopefully we'll start managing that better soon :)

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a244e3a on pvcarrera:session-host into e0ef26a on codebar:master.

despo added a commit that referenced this pull request Nov 9, 2013
@despo despo merged commit 0e6f301 into codebar:master Nov 9, 2013
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.

3 participants