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

質問の新規作成時にWIPを経由して公開すると「質問をWIPとして保存しました。」のメッセージが出たままになる #6517

Closed
ogawa-tomo opened this issue May 13, 2023 · 8 comments
Assignees
Projects

Comments

@ogawa-tomo
Copy link
Contributor

概要

タイトルの通りです。
ユーザーとしては、「あれ?質問が公開できていないのかな?」と思ってしまう可能性があると思います。

再現手順

  1. 質問作成ページ(questions/new)で適当に質問を作成し、「WIP」をクリックして保存
  2. 「質問をWIPとして保存しました。」というフラッシュメッセージが出て、質問ページに遷移することを確認
  3. 「内容修正」をクリック
  4. 質問編集ページに自動で遷移し、「質問をWIPとして保存しました。」というフラッシュメッセージが出たままであることを確認
  5. 適当に内容を編集し、「質問を公開」をクリック
  6. 質問ページに自動で遷移し、「質問をWIPとして保存しました。」というフラッシュメッセージが出たままであることを確認

期待される振る舞い

上記再現手順の4の時点で、フラッシュメッセージは消すべきだと思われます。

スクリーンショット

image

環境

  • OS: Windows10
  • ブラウザ: chrome
  • バージョン: 113.0.5672.93(Official Build)

原因

フラッシュメッージが表示されたままになってしまうのは、保存済みの質問の編集操作はapp/controllers/questions_controller.rbが担っており、ブラウザとしてはページ遷移せずにJavascriptで操作しているためと思われます。

@komagata komagata added this to いつかやる in 開発 May 13, 2023
@machida machida moved this from いつかやる to やる in 開発 May 15, 2023
@machida machida moved this from やる to 今のイテレーション in 開発 May 15, 2023
@ogawa-tomo ogawa-tomo moved this from 今のイテレーション to 作業中 in 開発 May 20, 2023
@ogawa-tomo
Copy link
Contributor Author

@komagata @machida
本イシューの仕様について相談させてください:bow:

上記イシュー本文では、「再現手順の4の時点で、フラッシュメッセージは消すべき」と書きました。
ただ、この部分の画面遷移はVueコンポーネントによって制御されているため、画面上部に表示されているフラッシュメッセージを消すことは難しいように思われました。Vueコンポーネントの管轄外であるためです。

そこで、質問の作成フローにおいては画面上部のフラッシュメッセージは使用せず、「内容変更」ボタン下部(Vueコンポーネント内)にメッセージを表示するよう統一したいと思うのですが、いかがでしょうか?
現状で既に、既存の質問を更新した際のメッセージはそこに表示されるようになっています。

具体的には、下記のような挙動に変更したいと考えています。ご確認お願いいたします。

質問を新規作成してWIPとして保存しそのまま公開するとき

1. 質問作成ページ/questions/newで質問を適当に作成し、「WIP」をクリック

修正前: 「質問をWIPとして保存しました。」のメッセージが画面上部に表示される
image

修正後: 「質問をWIPとして保存しました。」のメッセージが「内容修正」ボタンの下部に表示される
image

2. そのまま「内容修正」→「質問を公開」をクリック

修正前: 「質問を更新しました。」のメッセージが「内容修正」ボタンの下部に表示され、「質問をWIPとして保存しました。」のメッセージが画面上部に表示されたままとなっている
image

修正後: 「質問を更新しました。」のメッセージが「内容修正」ボタンの下部に表示される
image

質問を新規作成してそのまま公開するとき

質問作成ページ/questions/newで質問を適当に作成し、「登録する」をクリック

修正前: 「質問を作成しました。」のメッセージが画面上部に表示される
image
修正後: 「質問を作成しました。」のメッセージが「内容修正」ボタンの下部に表示される
image

@komagata komagata added the 1 label May 24, 2023
@ogawa-tomo
Copy link
Contributor Author

5/24 開発ミーティングメモ
全てトーストに統一という方針はありうる。
「担当者になりました」「コメント投稿」などはトーストを使っているはず。

→駒形さんが改めて検討したうえで後日回答していただくことになった。

@komagata
Copy link
Member

komagata commented May 24, 2023

@ogawa-tomo 仕様を考えました。

アドホックな対応になりますが、一時的な対応ということで、

「変更時にflashメッセージを直接書き換える」

ということでいきたいと思います。

document.querySelector('#flash').innerHtml = '質問を公開しました。'

的な感じで大丈夫です。(厳密にコードは違うものになると思いますが)

根本的には「Q&Aの編集ページはSPA的なものじゃなくする(元に戻す)」とかが良いと思っていますが今回はアドホックな対応でお願いします。

@ogawa-tomo
Copy link
Contributor Author

承知しました、ありがとうございます!

@ogawa-tomo
Copy link
Contributor Author

@komagata @machida こちらの本番環境動作確認について質問をさせてください。こちらは質問の新規作成時の挙動に関するイシューですが、動作確認にあたり本番環境で実際に質問を作成してよいでしょうか?

@machida
Copy link
Member

machida commented Jun 19, 2023

@ogawa-tomo はい!通知はメンターにしか飛ばないので確認用の質問を作って大丈夫ですー🙆‍♂️

@ogawa-tomo
Copy link
Contributor Author

承知しました、ありがとうございます!

@ogawa-tomo
Copy link
Contributor Author

本番環境で動作確認ができたのでクローズします。

開発 automation moved this from レビュー中 to 完成 Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
開発
  
完成
Archived in project
Development

No branches or pull requests

3 participants