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

休会からの復帰機能を追加 #5369

Merged
merged 31 commits into from
Nov 8, 2022
Merged

休会からの復帰機能を追加 #5369

merged 31 commits into from
Nov 8, 2022

Conversation

komagata
Copy link
Member

@komagata komagata commented Aug 14, 2022

ログイン画面に「休会からの復帰」リンクを追加。

スクリーンショット 2022-08-14 22 07 49

休会している人がログインしようとした時のエラーメッセージを追加。(kyuukaiというユーザーが休会しているユーザーとして登録済みです)

スクリーンショット 2022-08-14 22 09 32

退会した人がログインしようとした時にエラーメッセージを修正。(yameoというユーザーが退会しているユーザーとして登録済み)

スクリーンショット 2022-08-14 22 13 47

休会からの復帰ページ追加。(kyuukaiというユーザーが休会しているユーザーとして登録済みです)

スクリーンショット 2022-08-14 22 16 06

休会してるユーザーのプロフィールに休会情報を追加。(複数回休会できるのでその情報を表示)

スクリーンショット 2022-08-14 22 22 23

@komagata komagata self-assigned this Aug 14, 2022
@komagata
Copy link
Member Author

@machida デザインお願い致します〜。
トップページのどこかに「休会からの復帰」ページへのリンクが必要かな〜と思うのでいい感じのところがあったらお願い致します!

@machida machida self-assigned this Aug 14, 2022
@machida
Copy link
Member

machida commented Aug 14, 2022

@komagata 了解ですー🙆‍♂️

@machida
Copy link
Member

machida commented Aug 16, 2022

@komagata トップページのフッタにリンクを貼り、全体的にちょっと整えました。

以下、バグかもしれのでご確認お願いします。

  1. loginページに行く(このときの URL は /login)
  2. バリデーションエラーでログインに失敗した(このときの URL は /user_sessions)
  3. /user_sessions でページをリロードするとエラーが出る

@machida machida removed their assignment Aug 16, 2022
@komagata
Copy link
Member Author

komagata commented Aug 16, 2022

@machida

リンクありがとうございます!

/user_sessions でページをリロードするとエラーが出る

これはどういったエラーが出ますか?

@machida
Copy link
Member

machida commented Aug 17, 2022

@komagata

このエラーでした。

image

エラーの出る条件がもっと入り組んでいたので、もう一度書き直します。

  1. ログインページに行き、kyuukai でログインしようとする
  2. 休会中なので復帰ページから復帰申請して、というメッセージがでる
  3. 復帰ページに行かず、ログインページにとどまったまま、今度はログインフォームを空欄のままサブミットボタンをクリック
  4. /user_sessions に遷移され、先程のエラーの画面が表示される

こんな操作をする人は滅多にいないとは思うので、修正にコストがかかりそうでしたら後での対応で大丈夫です〜


もう一つ、同じエラーが出るケースを見つけました。

  1. ログインページに行き、kyuukai でログインしようとする
  2. 休会中なので復帰ページから復帰申請して、というメッセージがでる
  3. 復帰ページに行かず、ログインページにとどまったまま、今度はユーザーをmachidaにして、パスワードも入力してサブミットボタンをクリック
  4. machida ではログインされず、/user_sessions に遷移され、先程のエラーの画面が表示される

@komagata
Copy link
Member Author

@machida ありがとうございます、修正いたします!

@komagata
Copy link
Member Author

@machida 上記、修正いたしました〜。

@machida
Copy link
Member

machida commented Aug 17, 2022

@komagata ありがとうございます🙏

@machida
Copy link
Member

machida commented Sep 27, 2022

最新の main から rebase しました。

keiz1213 and others added 21 commits November 8, 2022 14:53
1. 分離せずにNotificationFacade経由で呼び出すと同じ通知が複数回discordに飛ぶため
存在しないJSを読み込んでいたので。
@komagata komagata merged commit 06e0d75 into main Nov 8, 2022
@komagata komagata deleted the feature/comeback branch November 8, 2022 07:10
@github-actions github-actions bot mentioned this pull request Nov 8, 2022
16 tasks
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.

3 participants