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

206: edit address #52

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hieuhv07
Copy link

Gemfile Outdated
@@ -15,7 +15,7 @@ gem "jbuilder", "~> 2.5"
gem "bootsnap", ">= 1.1.0", require: false
gem "slim-rails"
gem "devise"
gem "bootstrap", "~> 4.3.1"
gem "bootstrap", "~> 4.3", ">= 4.3.1"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tại sao lại sửa cái này?

Copy link
Author

Choose a reason for hiding this comment

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

Em chạy bundle hén lâu đó anh, em chờ cả mấy phút luôn mà cũng chưa xong, nên em chỉ định rõ phiên bản cho nó thì nó chạy đc anh

Copy link
Collaborator

Choose a reason for hiding this comment

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

chỉ định rõ phiên bản thì ghi mỗi số hiệu của phiên bản đó thôi, không ghi như thế này

Copy link
Author

Choose a reason for hiding this comment

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

dạ, để em update lại ạ

@hieuhv07 hieuhv07 force-pushed the 206/edit_address branch 2 times, most recently from 9a76bc1 to 96fadf5 Compare September 3, 2019 08:25
.gitignore Outdated
@@ -33,4 +33,5 @@ config/environments/*.local.yml

# Ignore application configuration
/config/application.yml

Copy link
Owner

Choose a reason for hiding this comment

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

Checkout file này

Gemfile.lock Outdated
@@ -126,7 +126,7 @@ GEM
dry-logic (~> 1.0, >= 1.0.2)
erubi (1.8.0)
execjs (2.7.0)
faker (2.2.0)
faker (2.1.2)
Copy link
Owner

Choose a reason for hiding this comment

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

Sao ko thay đổi Gemfile mà �Gemfile.lock lại thay đổi

Copy link
Author

Choose a reason for hiding this comment

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

tại vì lúc đầu em sửa gem 'bootstrap', '~> 4.3.1' thành gem 'bootstrap', '~> 4.3', '>= 4.3.1' , bị anh việt comment là là nếu chỉ định phiên bản thì không cần viết theo gem 'bootstrap', '~> 4.3', '>= 4.3.1', mà viết theo 'bootstrap', '~> 4.3.1' cũng được, nên em sửa rồi chạy bundle lại nên hén có thay đổi đó anh

Copy link
Owner

Choose a reason for hiding this comment

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

Thế thì checkout file này ra là đc mà

Copy link
Author

Choose a reason for hiding this comment

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

em đã checkout rồi anh

.modal-header
h4.modal-title = t ".edit"
= button_tag t(".btn"), class: "close", data: {dismiss: "modal"}, type: "button"
.modal-body
Copy link
Owner

Choose a reason for hiding this comment

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

Sao ko dùng chung modal với new address

Copy link
Author

Choose a reason for hiding this comment

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

tại vì edit address, em dùng form_tag, method: :patch và url của nó là url của edit, nên viết riêng ra luôn anh

Copy link
Owner

Choose a reason for hiding this comment

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

Em có thể dùng chung 1 modal sau đó dùng JS để thay đổi URL và method trong form sau mà

@hieuhv07 hieuhv07 force-pushed the 206/edit_address branch 2 times, most recently from 783eaa1 to fa32514 Compare September 3, 2019 13:33
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.

7 participants