Skip to content
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

バグ?貸出の編集画面で更新回数を編集できるようになっているが、できない #1005

Open
yegusa opened this issue Dec 15, 2015 · 5 comments
Labels
Milestone

Comments

@yegusa
Copy link

yegusa commented Dec 15, 2015

[貸出期間の延長] をクリック

image

貸出の編集画面が表示される (いじってない初期画面)

image

返却期限は2015-12-25から2015-12-30に変更、 更新回数は0のまま、[更新する]ボタンをクリックする

image

貸出期限が 2015-12-30 になっている、[編集]リンクをクリックする

image

編集画面が表示される

  • 新しい返却期限日:2015-12-25 ( 新しい返却期限ってなに?)
  • 返却期限:2015-12-30 (設定した日に代わっている)
  • 更新回数:1 (→ 0から1に代わっている)

image

編集画面で更新回数を0にして、[更新する]ボタンをクリック

image

編集画面 エラーがでてできない(更新回数が2に代わっている)

image

vagrant@vagrant-ubuntu-trusty-64:~/enju$ bundle list | grep enju

  • enju_biblio (0.1.1)
  • enju_circulation (0.1.0 afc4ce5)
  • enju_event (0.1.17 f886ee3)
  • enju_flower (0.1.0)
  • enju_leaf (1.1.1 917e0e9)
  • enju_library (0.1.0)
  • enju_manifestation_viewer (0.1.0)
  • enju_message (0.1.14)
  • enju_ndl (0.1.0)
  • enju_purchase_request (0.1.0 78c4280)
  • enju_subject (0.1.0)
@yegusa
Copy link
Author

yegusa commented Dec 19, 2015

#1009@masao と話し合ったところ

  • 「更新回数」の編集はできないようにする
  • 「更新回数を増やさない」というチェックできるようなチェックボックスをつける(デフォルトはチェックなし)
  • ↑のチェックが入っていた時は、更新回数を増やさないようにする

画面案
image

という開発をするのでいいのではないか ということになりました。

そうしておけば

  • 更新回数を増やさずに返却期限だけを編集することが可能になる
  • 更新回数を増やして返却期限を編集することも可能になる

となると思います。

結構重要な機能だと思いますので、1.2.1に入れておきます。

@yegusa
Copy link
Author

yegusa commented Dec 19, 2015

1.1系のマニュアルには更新回数は修正できない旨をかいておきました。

@yegusa
Copy link
Author

yegusa commented Dec 26, 2015

新しい案を考えてみました。

赤四角の部分を
image

以下のようにすればいいのじゃないかと思います。
image

  • 「貸出の更新」はUser権限のときに見えるもの
  • 「貸出の更新」と「貸出期限日の編集」はLibrarian権限以上の時に見えるもの

となっています。

@masao
Copy link
Member

masao commented Dec 26, 2015

全体によさそうな気はします。ただし、メッセージ・ラベルは英語にすることを考えると、文での表現にすると面倒そうです。

@yegusa
Copy link
Author

yegusa commented Dec 26, 2015

じゃあ、こんなかんじでどうでしょう

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants