search

SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

#200: Use select form to choose TA in admin #215

Closed skymountain closed 9 years ago

skymountain commented 9 years ago

担当 TA を select form で選択できるようにしました．

westpaddy commented 9 years ago

あと、admin一覧の取得apiがmaster.cgiでいいのかが疑問なのと、 selectCellで変更したときにリアクションがなにもなくてきちんと動いてるのかどうかが分かりにくいです。

skymountain commented 9 years ago

admin一覧(all_admins) は master.yml の情報なので master.cgi でいいんじゃないですかね．むしろ user/admin/token が master.cgi にあるのがおかしいと思います． selectCell は個人的には option を選ぶのが選択になってるのでこれで十分だと思ってますが，どうすればわかりやすくなりますか？

westpaddy commented 9 years ago

master.cgiはmaster.ymlにアクセスするためのapiだったんですね。 selectCellは少なくともリクエストが失敗したことを何らかの形で反映してほしいです。 あんまりよくない気がしますが、単純にやるとすると元のoptionに戻すとかですかね？

skymountain commented 9 years ago

初期の commit ba3a3080e1f17d1403fa1b76b00c1a84cf3c7d8c にその匂いを感じますね． select は少し考えます．

skymountain commented 9 years ago

編集ボタンとキャンセルボタンを付けました．

skymountain commented 9 years ago

size 属性を消すのを忘れてました．

skymountain commented 9 years ago

消しました．

westpaddy commented 9 years ago

更新されていないみたいです

skymountain commented 9 years ago

失礼，今度こそ． ちなみに TextCell と SelectCell はけっこう似た部分があるので共通化しようとしたけど，parent/child で行ったり来たりが起こってわかりにくくなりそうだったので共通化はやめました．