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

Develop #1

Closed
wants to merge 15 commits into from
Closed

Develop #1

wants to merge 15 commits into from

Conversation

u-sho
Copy link
Member

@u-sho u-sho commented May 20, 2024

@u-sho
Copy link
Member Author

u-sho commented May 31, 2024

image

@u-sho u-sho marked this pull request as ready for review May 31, 2024 01:16
@u-sho u-sho requested a review from takahash May 31, 2024 01:17
@u-sho
Copy link
Member Author

u-sho commented May 31, 2024

問題

メモ:

  • もしかして firebaseServerApp でセッション管理すると middleware.ts での route protection がうまくいくかも(あえて use server でやったほうが楽?)
  • firebase UI は v9 にまだ対応していません (cf. Support Firebase JS SDK v9 and v10 firebase/firebaseui-web#1055 )
  • 作っているうちに慣れてきてしまいましたが配色が微妙かも
  • ログインページと新規登録ページ統合したほうがいいかも(<form>内で<button type="submit">が複数あるとinvalidなのでわけています)

@takahash
Copy link
Member

takahash commented May 31, 2024

@u-sho

  • 難しくなるので middleware.ts を使わず、 layout.tsx などクライアントサイドで対応してください
  • firebase UIは使わなくて良いので、自前でログインフォームを用意してください
  • 配色についてもう少し落ち着いた感じ(例えばGitHubくらい)にしてほしいです

@u-sho
Copy link
Member Author

u-sho commented May 31, 2024

  • layout.tsx ('use client') で非ログイン時のルート遷移を行うようにしました

  • 自前ログインフォームです
    image

  • 配色考えるのが難しかったので一旦GitHubの色を使って落ち着かせました

image
image

@takahash takahash closed this Jul 11, 2024
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.

2 participants