Skip to content

Conversation

@Kosei805
Copy link
Contributor

やったこと

  • やったこと (コミットのハッシュ)
  • カートページへのリダイレクトの実装 7491425
  • loaderでユーザー情報を取得するよう書き換え 5f5efe9
  • ↑をsessionでそもそもユーザー情報を持つよう書き換え f21e84d

確認した方法

pnpm run devで確認

スクリーンショット

  • カートへのリダイレクト
2024-11-14.13.51.26.mov
  • ログインとログアウト
2024-11-14.13.53.08.mov

自動生成したコード

  • ファイル名

@Kosei805 Kosei805 added the frontend frontend development label Nov 14, 2024
@Kosei805 Kosei805 requested a review from kimurash November 14, 2024 04:57
@Kosei805 Kosei805 self-assigned this Nov 14, 2024
if (navigation.state === 'idle') {
if (userId) {
if (!!userData) {
// CookieにユーザIDが存在する
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントも変えておこう

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメント変えました

Comment on lines 108 to 109
// 状態変数にユーザ情報が保存されていない
// ユーザ情報を取得するAPIを呼び出す
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントも変えておこう

Copy link
Contributor Author

@Kosei805 Kosei805 Nov 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメント変えました

interface SessionData {
userId: string;
user: User;
sessionToken: string;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

userの中にsessionTokenも含まれてるからsessionTokenは消しておこう

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

消しました

@kimurash
Copy link
Member

この会話が解決してからApproveを出します

#135 (comment)

@Kosei805
Copy link
Contributor Author

この会話が解決してからApproveを出します

#135 (comment)

チェックよろしく

@Kosei805 Kosei805 merged commit f519027 into main Nov 14, 2024
3 checks passed
@Kosei805 Kosei805 deleted the 66-cart-page-redirect branch November 14, 2024 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend frontend development

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

カート追加時にカートページにリダイレクト

3 participants