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

管理画面 コメント一覧のajax_* 系JSで、エラーハンドリングが不足している #140

Open
1 of 3 tasks
uzulla opened this issue Sep 14, 2020 · 1 comment
Assignees

Comments

@uzulla
Copy link
Collaborator

uzulla commented Sep 14, 2020

  • sigが不正だった場合に正しくユーザーがリカバリーするフローが存在しない
    • 画面をリロードすると治るが、ユーザーには理由が通知されないので、通知してリロードをかける
    • (sigは他の画面でも変わるので、ちょっとajaxと相性が悪いかも) sigはログイン時生成で固定になった。
  • Javacsript(jquery.ajax)のコードが素朴で、エラーハンドリングが記述されていない
    • エラーが通知されないので、失敗時にユーザーがまちつづけることになる
      • 「予期せぬ」レスポンスを受け取ったとき
      • ステータスコード200以外が帰った時
      • json以外が帰った時、等。、
  • Ajax系のE2Eテストを作成する

  • ajax時のリカバリされないエラー時に通知をだすようにした
  • Ajax系のE2Eテスト(範囲が広いため、長期になるかもしれない)
  • sig不正時のエラー処理
@uzulla uzulla changed the title 管理画面 コメント一覧のajax_* 管理画面 コメント一覧のajax_* 系JSで、エラーハンドリングが不足している Sep 14, 2020
@fc2dev
Copy link
Contributor

fc2dev commented Oct 5, 2020

こちら、ご対応お願いいたします。

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

No branches or pull requests

2 participants