search

SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Issue email #111

Closed krtx closed 9 years ago

krtx commented 9 years ago

user.cgi に email をデータに含めるかどうかのオプションを追加しました。とりあえず admin/ の画面でだけ email が含まれるようになっています。

krtx commented 9 years ago

TODO: not su ならば email を見れないようにする

westpaddy commented 9 years ago

とりあえず緊急パッチ的な意味ではいいと思うんですが、 要求仕様がやはりよくわからないので、この方向性でいいのかという疑問があります

krtx commented 9 years ago

方針としては email の不要なやりとりを避けるということだと思ってたのですが、そういうことではない？

westpaddy commented 9 years ago

必要最低限の情報だけやりとりする。ということでいいですか？

krtx commented 9 years ago

yes

westpaddy commented 9 years ago

TODO の not su なら email 見れないようにするっていうのはいらない気がする。

westpaddy commented 9 years ago

とりあえずこれは入れます。

skymountain commented 9 years ago

not su なら email は見れないはいる気がします．(そうでないとこのオプションとユーザーIDがわかっていればメールアドレス情報がとれてしまう)