お問い合わせフォームの実装 #148

Open
wants to merge 10 commits into
from

Projects

None yet

3 participants

@HISAS
Contributor
HISAS commented Dec 14, 2016

No description provided.

Hisashi Kame... and others added some commits Dec 12, 2016
@komagata komagata temporarily deployed to interns-com-pr-148 Dec 14, 2016 Inactive
@HISAS HISAS added the in progress label Dec 14, 2016
@HISAS HISAS changed the title from Contact form to お問い合わせフォームの実装 Dec 14, 2016
@HISAS HISAS requested a review from komagata Dec 14, 2016
@HISAS HISAS referenced this pull request Dec 14, 2016
Closed

フォームの実装 #145

@HISAS
Contributor
HISAS commented Dec 14, 2016

@komagata レビューお願いします。

@komagata
Member

@HISAS circleciがコケているので確認して下さい。

@komagata komagata temporarily deployed to interns-com-pr-148 Dec 16, 2016 Inactive
@HISAS
Contributor
HISAS commented Dec 16, 2016

@komagata CI通したのでご確認お願いします。

@machida machida added the review label Dec 16, 2016
app/models/contact.rb
+
+ validates :name, presence: true
+ validates :name_phonetic, presence: true
+ validates :email, presence: true, uniqueness: true
@komagata
komagata Dec 16, 2016 Member

状況が変わった人が2度目の応募をすることがあるのでuniqじゃなくて良いです。

config/routes.rb
@@ -38,6 +38,6 @@
get 'login' => 'user_sessions#new', as: :login
get 'logout' => 'user_sessions#destroy', as: :logout
- #TODO 仮のお問い合わせページURL(デザインのみシステム未実装、実装後削除)
- get "/contact", to: "home#contact"
+ get "contacts/new", to: "contacts#new"
@komagata
komagata Dec 16, 2016 Member
resource :contact, only: %i(new create)

みたいな感じがいいです。

app/controllers/contacts_controller.rb
+ @contact = Contact.new(contact_params)
+
+ if @contact.save
+ redirect_to root_path, notice: t('contact_was_successfully_created')
@komagata
komagata Dec 16, 2016 Member

応募があったらinfo@fjord.jpにメールが来てほしいです。
あと、管理画面も欲しいっすね。

@komagata
komagata Dec 26, 2016 Member

新しいデザインを早く反映させたいので管理画面は後でいいや。

@HISAS
Contributor
HISAS commented Jan 11, 2017

@komagata 修正しましたのでレビューお願いします。

@komagata

管理画面がないのであれが、メールで応募内容が知りたいです。

現状の応募フローを参考にしてください。

@HISAS
Contributor
HISAS commented Jan 13, 2017

@komagata 応募フローですがどこを見て参考にすればよいでしょうか…?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment