Skip to content

取引相手確定処理停止#299

Open
hatimitusiropp wants to merge 5 commits intomasterfrom
d_giveme-from-others-list
Open

取引相手確定処理停止#299
hatimitusiropp wants to merge 5 commits intomasterfrom
d_giveme-from-others-list

Conversation

@hatimitusiropp
Copy link
Collaborator

themes/freecycle/members/singleにあったgiveme-from-others-list.phpを
themes/freecycle/members/single/backup_pageに移動しました。

@hashikuchi
Copy link
Owner

「取引相手確定処理の停止」なのに、「他の人からのください」ページをまるごと消してしまっている意図を図りかねています。
どういう経緯と理由でこのような実装になったのでしょうか?

@hatimitusiropp
Copy link
Collaborator Author

取引相手を決める時に、そのページで取引相手を選んでいたのでそのページを消せばいいのかなと考えてしまっていました

@hashikuchi
Copy link
Owner

なるほど。それはやりすぎですね〜

実装する際は、影響範囲を必要最低限に留めることを意識しましょう。
また、使用者目線から見てわかりやすいかどうかも考えましょう。
今回の例だと、ページが消えているのに「他の人からのください」タブは残っているので、タブを選んでも何も表示されない、という状況になっており、これはユーザに対して不親切です。

個人的には、ください確定ボタンのdisable属性を変えて押せないようにして、ひとこと「確定処理は停止中です」と画面上に書いておけばいいのではないかと思います。

また必要になったらすぐ使えるようにできるしね。ちょっと考えてみてください。

@hatimitusiropp
Copy link
Collaborator Author

なるほど!たしかにそうですね、やりすぎました汗
影響が少なくなるように考えてみます

@hatimitusiropp
Copy link
Collaborator Author

giveme-from-others-list.phpとsingle.phpの一部を切り取ってbackup_pageに入れました。
「くださいリクエスト一覧」の下と、商品ページの編集ボタンの近くに”現在、ユーザ間での取引は停止されています”というコメントをのせておきました。

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ログインユーザが誰かにかかわらず、常に編集ボタンが出るようになっていませんか?

@hatimitusiropp
Copy link
Collaborator Author

すいません、出品していないユーザには表示されないようにしました。

@hatimitusiropp
Copy link
Collaborator Author

あ、これだと取引完了した本にも編集ボタンがついちゃいますね汗
もう一回直します。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments