-
Notifications
You must be signed in to change notification settings - Fork 5
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
base: develop
Are you sure you want to change the base?
Conversation
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" |
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.
Tại sao lại sửa cái này?
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.
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
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.
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
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.
dạ, để em update lại ạ
9a76bc1
to
96fadf5
Compare
.gitignore
Outdated
@@ -33,4 +33,5 @@ config/environments/*.local.yml | |||
|
|||
# Ignore application configuration | |||
/config/application.yml | |||
|
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.
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) |
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.
Sao ko thay đổi Gemfile mà �Gemfile.lock lại thay đổi
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.
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
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.
Thế thì checkout file này ra là đc mà
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.
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 |
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.
Sao ko dùng chung modal với new address
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.
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
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.
Em có thể dùng chung 1 modal sau đó dùng JS để thay đổi URL và method trong form sau mà
783eaa1
to
fa32514
Compare
fa32514
to
3687a31
Compare
http://192.168.100.19:3333/issues/206