Skip to content

Chapter 13#10

Open
levanluu wants to merge 1 commit intomasterfrom
chapter_13
Open

Chapter 13#10
levanluu wants to merge 1 commit intomasterfrom
chapter_13

Conversation

@levanluu
Copy link
Owner

@levanluu levanluu commented Oct 1, 2020

No description provided.

@levanluu levanluu force-pushed the chapter_13 branch 5 times, most recently from 166816b to df14892 Compare October 5, 2020 09:59
@levanluu
Copy link
Owner Author

levanluu commented Oct 5, 2020

ready

end

def destroy
@micropost.destroy

Choose a reason for hiding this comment

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

nếu xóa fail thì sao

@@ -0,0 +1,2 @@
module MicropostsHelper

Choose a reason for hiding this comment

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

xóa file

@@ -0,0 +1,2 @@
module PasswordResetsHelper

Choose a reason for hiding this comment

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

xóa file

end

def feed
Micropost.where("user_id = ?", id)

Choose a reason for hiding this comment

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

viết scope cho where nè

config/routes.rb Outdated
@@ -1,4 +1,6 @@
Rails.application.routes.draw do
get 'password_resets/new'
get 'password_resets/edit'

Choose a reason for hiding this comment

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

xóa 2 line này

@levanluu
Copy link
Owner Author

levanluu commented Oct 5, 2020

ready


def destroy
if @micropost.destroy
flash[:success] = t("micropost.micropost_deleted")

Choose a reason for hiding this comment

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

indent

@@ -0,0 +1,7 @@
require 'test_helper'

Choose a reason for hiding this comment

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

Xóa file

@vanvtt-0952
Copy link

merged
fix comment đi hì

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