search

Recursion-Group-K / sketch

https://sketch-frontend-d11d98fe289b.herokuapp.com/
0 stars 0 forks source link

Feat/auth/signup #89

Closed tkwonn closed 2 years ago

tkwonn commented 2 years ago

85

一度ローカルにDevelopの最新を持ってきて、feat/auth/signupブランチを切って作業を行うべきところ、ローカルのDevelopで作業を進めてしまったせいで、コミットメッセージと内容がマッチしていないです。申し訳ありません。

行ったこと

簡単なローディングとエラー文の表示 success -> Drawingページへのページ遷移

質問

signup時にはアドミン権限でUserを作成しPOSTリクエストを送る処理のみで大丈夫でしょうか？ 少し、急ぎめでやってしまったので、抜け漏れがあったらすみません！

mcnLeandro commented 2 years ago

@tkwonn ありがとうございます！ ユーザーの作成は完璧にできてました！

できてなかったのは以下です！

ということなので、どちらでも問題ないのですが、 先に #74 やるかこちらやるかだけ教えていただけると進捗把握できるのでありがたいです🙏

tkwonn commented 2 years ago

先にこっちやってから #74 やります！

tkwonn commented 2 years ago

@mcnLeandro

できてなかったのは以下です！ エラーの表示 そのままlogin

こちら修正したので、レビューお願いします！

やったこと

ユーザネームとメールアドレスそれぞれの下にエラー文が表示されるようにしました。 また成功時はステータスコードではなく、Userのインスタンスが返されていたので、それをチェックしてMutationにてSIGNUP SUCCESSを行うようにしました。

ページのスタイリングもちょこっとだけ修正しました。