-
Notifications
You must be signed in to change notification settings - Fork 382
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
First do it #60
First do it #60
Conversation
@@ -23,3 +23,6 @@ npm-debug.log | |||
|
|||
# Ignore bundle dependencies | |||
vendor/ruby | |||
|
|||
# RVM gemset | |||
.ruby-gemset |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious if this should be project specific.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I keep all gems in isolated gemsets for two reasons:
- I maintain
yo
generator, so I scaffold a lot for testing purposes. It's easier for me to delete scaffolded test gems by deleting gemset. - To sleep easily.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should .ruby-gemset be in developer's global gitignore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should .ruby-gemset be in developer's global gitignore?
If everybody is using gemset-per-app strategy, then no. Otherwise — yes.
@alexfedoseev Looks AWESOME. I'll let you merge this change into the PR on my repo. Please respond to my comments here, however. Please merge --squash your changes on top of mine. |
Squash merge of #60 from @alexfedoseev (thanks!) 1. What should we name the reducer? Seems confusing in terms of how the reducer name gets put as part of the state. 2. Functional issue in that we don't clear out the last name/commment after ajax is successful. Seems that we would need to have the component state inside of the main store. 3. Ajax counter to show busy indicator of async in progress might not be working. 4. We should show a simple validation on the server and the display of a simple error message on the form.
Squash merge of #60 from @alexfedoseev (thanks!) 1. What should we name the reducer? Seems confusing in terms of how the reducer name gets put as part of the state. 2. Functional issue in that we don't clear out the last name/commment after ajax is successful. Seems that we would need to have the component state inside of the main store. 3. Ajax counter to show busy indicator of async in progress might not be working. 4. We should show a simple validation on the server and the display of a simple error message on the form.
Already merged to other PR. Closing. |
Squash merge of #60 from @alexfedoseev (thanks!) 1. What should we name the reducer? Seems confusing in terms of how the reducer name gets put as part of the state. 2. Functional issue in that we don't clear out the last name/commment after ajax is successful. Seems that we would need to have the component state inside of the main store. 3. Ajax counter to show busy indicator of async in progress might not be working. 4. We should show a simple validation on the server and the display of a simple error message on the form.
Squash merge of shakacode/react-webpack-rails-tutorial#60 from @alexfedoseev (thanks!) 1. What should we name the reducer? Seems confusing in terms of how the reducer name gets put as part of the state. 2. Functional issue in that we don't clear out the last name/commment after ajax is successful. Seems that we would need to have the component state inside of the main store. 3. Ajax counter to show busy indicator of async in progress might not be working. 4. We should show a simple validation on the server and the display of a simple error message on the form.
Squash merge of shakacode/react-webpack-rails-tutorial#60 from @alexfedoseev (thanks!) 1. What should we name the reducer? Seems confusing in terms of how the reducer name gets put as part of the state. 2. Functional issue in that we don't clear out the last name/commment after ajax is successful. Seems that we would need to have the component state inside of the main store. 3. Ajax counter to show busy indicator of async in progress might not be working. 4. We should show a simple validation on the server and the display of a simple error message on the form.
Now it renders the view, but we should discuss a lot of stuff (: