Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft initial update for Rush Hour readme #1221

Merged
merged 3 commits into from Oct 5, 2016
Merged

Conversation

s-espinosa
Copy link
Member

@case-eee Draft of the updated Rush Hour readme based on the comments on the M2 Trello board. Items outstanding include:

  • Clarify expectations for ActiveRecord methods on models
  • Add explanation to Iteration 1 to give more direction on what they should break out

I wasn't sure what we wanted to do on the first one, and wanted to make sure we were on the same page about how much to give on the second. Let's discuss!

@@ -179,6 +190,9 @@ We already have a `PayloadRequest` model and database table, and we know that a

A Client has two attributes, an `identifier`, and a `rootUrl`.

* **identifier** is skin to a username (e.g. `jumpstartlab`). This will be unique in the universe of our app (no two clients should have the same identifier).
Copy link
Contributor

Choose a reason for hiding this comment

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

"akin" instead of "skin"

@case-eee
Copy link
Contributor

case-eee commented Oct 4, 2016

@s-espinosa I made one small spelling error comment - once you fix this, go ahead and merge. I'm going to take a stab at adding some more clarification to this tonight

@s-espinosa s-espinosa changed the title [WIP] Draft initial update for Rush Hour readme Draft initial update for Rush Hour readme Oct 5, 2016
@s-espinosa s-espinosa merged commit de61632 into master Oct 5, 2016
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.

None yet

2 participants