-
Notifications
You must be signed in to change notification settings - Fork 71
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
【メンター向け機能】日報にコメントをだけ入れて確認済みにしなかったらアラートを出す #6928
Comments
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
📝 コメントのreact化が完了したらスタート。 |
@machida さん 上のコメントで
とありますが、該当しそうなIssueを確認したところまだ作業中のようでした。 本Issueは作業を進めてよいのでしょうか? |
@unikounio ごめんなさい!これを見逃してました。別のIssueをアサインします🙇♂️ |
@machida さん |
@unikounio こちらアサインを外してやるに戻させていただきます🙇♂️ |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
@su-su-su-su すいません、React化はまだ先になりそうなので、着手をお願いしますー🙏 |
@machida |
お疲れ様です。 window.addEventListener('beforeunload', this.handleUnload)
handleUnload(e) {
const userCommented = this.comments.some(
(comment) => comment.user.id === this.currentUserId
)
const noConfirmed = !this.checkId;
if (userCommented && noConfirmed) {
e.preventDefault()
e.returnValue = '日報を確認済みにしていませんがよろしいですか?'
}
} 調べた結果、ブラウザの仕様により またコメント途中にページを離れる際に出るアラートの部分(app/javascript/warning.js)も調べたのですが、こちらもカスタムメッセージがあるものの表示されるのは標準の警告メッセージが表示されてしまいます。 この問題に対して、カスタムメッセージを表示するための他のアプローチやブラウザの標準メッセージを利用するなど、何か他に解決方法がありましたら、ご教示いただけると幸いです。よろしくお願いいたします。 |
@su-su-su-su 「ページを離れる際に」だと、デフォルトのメッセージしか表示できなくなってしまったので、 メンターのコメント欄は以下のようになってますが、 確認済になっていないとき、Bではなく、Aをクリックしたら、 「日報を確認済みにしていませんがよろしいですか?」 というメッセージが表示される、 というようにしたいです。 |
ありがとうございます。画像も添付して頂き助かります!
承知しました。そのように変更したいと思います。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
機能の説明
メンターが日報にコメントをだけ入れて確認済みにしないまま画面を離れようとしたらアラートを出す。
メッセージ例「日報を確認済みにしていませんがよろしいですか?」
何故この機能が必要なのか
本当はコメントを入れて日報を確認済みにすべきところを、コメントだけ入れて画面を離れてしまうことがたまにあるから。
ただし、確認済みにしなかったとしても致命的な問題ではないので、そこまで優先度の高い機能ではないかもしれない。
The text was updated successfully, but these errors were encountered: