-
Notifications
You must be signed in to change notification settings - Fork 0
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
second commit #1
base: master
Are you sure you want to change the base?
Conversation
app/controllers/tasks_controller.rb
Outdated
@@ -0,0 +1,56 @@ | |||
class TasksController < ApplicationController | |||
def index | |||
puts "called TasksController#index" |
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.
@Takuma1111
不要なデバッグコードは削除しちゃいましょう!
app/controllers/tasks_controller.rb
Outdated
else | ||
flash.now[:alert] = @task.errors.full_messages | ||
render :index | ||
end |
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.
@Takuma1111
このへんのインデントも修正お願いします!
app/controllers/tasks_controller.rb
Outdated
end | ||
|
||
def show | ||
@task = Task.find(params[:id]) |
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.
@Takuma1111
before_action使って処理をまとめてみて下さい!
app/controllers/tasks_controller.rb
Outdated
|
||
@tasks = Task.all | ||
render :index | ||
end |
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.
@Takuma1111
このへんもインデント修正お願いします!
app/views/tasks/index.html.erb
Outdated
<% end %> | ||
</ul> | ||
|
||
________________________________________________ |
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.
@Takuma1111
線を引く時は<hr>
使いましょう!
@Takuma1111 |
@kazu-taka befor_actionと 使いました! controllerは全体的に間隔あけたほうがいいですかね |
@Takuma1111 |
@Takuma1111 |
No description provided.