SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Mail configuration #248

Closed weedssoul closed 9 years ago

weedssoul commented 9 years ago

Fix #220 .

メールサーバを設定できるようにしました。 config/master.yml に

mail:
  :address: "smtp.example.com"
  :port: 587
  :domain: "example.com"
  :authentication: "plain"
  :user_name: "user_name"
  :password: "password"

という設定を追加すれば、そのサーバを使って送信できるようにしました。

正確に問題を理解できてるかわからないので、指摘の方あればどんどんお願いします。

chiro commented 9 years ago

lib/reset.rb にもメールを送ってる箇所があるので、そこもお願いします。 :pray:

chiro commented 9 years ago

master.yml の形式が変わったので、 config/schema/master.yml も変えなきゃいけないかも……?

chiro commented 9 years ago

あと、 config/example/master.yml に例を足してほしいです。 いくつもすいません :sweat_drops:

weedssoul commented 9 years ago

reset.rb のほうもメールサーバの設定を使うようにしました。

また、master.yml で、ハッシュのキーをシンボルにしてたのですが、 それを文字列に変更し、 reset.rb と register.rb の内部で 文字列からシンボルに変換するようにしました。

schema/master.yml に mail を追加しました。 他にも使える option はあるのですが、とりあえず必要そうなものだけ書いてます。

chiro commented 9 years ago

どうも、ありがとうございます。 :sparkles: よさそうに思います。 :+1:

chiro commented 9 years ago

@krtx 一応見てもらえますか?

krtx commented 9 years ago

良いと思います