Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Updates to Preview Branch (feature/create-companies-list) ↗︎
Tasks are run on every commit but only new migration files are pushed.
View logs for this Workflow Run ↗︎. |
|
@kazuma-naka さん、 |
|
@Jyoumama さん、 お疲れ様です。 supabase を起動時に seed.sql を読み込むと以下の SQL が実行されるという認識を私は持っています。 @Jyoumama さんのスクショのデータが 日報を拝読しまして Companies の一覧が表示されない理由は RLS が関係していますでしょうか？ INSERT INTO companies (name, website, memo) VALUES ('株式会社ロッカ', 'https://lokka.jp', '良い会社');
INSERT INTO companies (name, website, memo) VALUES ('株式会社リンゴ', 'https://example.com/apple', 'リンゴは良い会社');
INSERT INTO companies (name, website, memo) VALUES ('株式会社バナナ', 'https://example.com/banana', 'バナナは良い会社');
INSERT INTO companies (name, website, memo) VALUES ('株式会社オレンジ', 'https://example.com/orange', 'オレンジは良い会社');
|
|
@kazumanさん、確認＆コメントをいただいてありがとうございます！ とり急ぎ、返信いたします。結論、これからこちらのプルリクエストにkazumaNさんが適用されている supabase db reset
supabase db seed私が入力した企業のダミーデータがkazumaNさんのローカル環境でご覧になれない理由は、私が |
連絡はこちらで大丈夫ですよ！ |
|
@kazuma-naka さん、お疲れ様です。 1.企業一覧のコンポーネント 動作確認時のご注意事項 git fetch origin feature/create-companies-list
git checkout feature/create-companies-listなお、ご存知かと思いますが、 supabase db seed実は、私のローカル環境で そこで、ローカル環境のsupabaseのRLSポリシーの設定に”アプリにデータ表示ができる様になった設定”を追加し、変更しました。そのmigrationファイルの変更（RLSポリシーの変更）を適用するために、以下のコマンドで実行し、最新のマイグレーションをローカル環境に適用しました。 supabase db pushその後、再度、 上記、分かりにくかったり（自分の実施したことが間違っていたりもするかもですし。。。） |
There was a problem hiding this comment.
この POLICY ですと認証されたユーザー、 未認証のユーザー関係なくすべてのユーザーが companies テーブルにアクセスできると思います。
会社一覧を参照する際にログインが必要でしたので気になりました。
どのユーザーが companies テーブルに何の操作できるのか、定義してあげる必要があると感じました。
There was a problem hiding this comment.
https://supabase.com/docs/guides/database/postgres/roles
こちらに supabase の roles の一覧があります
Jyoumama
force-pushed
the
feature/create-companies-list
branch
from
6bf1118 to
6330bd6
Compare
|
@kazuma-naka さん、 🛠 主な変更点
👀 レビューしてほしい点
お手すきの際に、レビューよろしくお願いします！🙏 |
There was a problem hiding this comment.
const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL || "http://127.0.0.1:54321";
もし process.env.NEXT_PUBLIC_SUPABASE_URL が設定されていないと http://127.0.0.1:54321 を使用しようとするのであまり必要のない定義に感じました。
There was a problem hiding this comment.
package.jsonのlocalとremoteがなぜ、分かれているのか？のコメント欄で記載したことと関連していますが、
自分のPC上では、env.localとenv.remoteで環境変数を分けています。
よって、こちらhttp://127.0.0.1:54321/ をわざわざ設定するようにしていました。
おっしゃる通り、環境変数が常に設定される運用なら、デフォルトの "http://127.0.0.1:54321" は不要で、環境変数が未設定ならエラーを投げるように設定していますし、http://127.0.0.1:54321/を削除しても動作が変わらなかったので削除しました！
src/components/Companies.tsx
Outdated
There was a problem hiding this comment.
Test は別ファイルに記述するとよいと思います
There was a problem hiding this comment.
Companies.test.tsxを作成し、テストは別に記述しました。
やっとパスしました〜。
There was a problem hiding this comment.
handleLogin() を使って init()されています。
ローカルのテストユーザーでログインをされているのですが必要がないと思うのですがいかがでしょうか？
There was a problem hiding this comment.
こちらの local と remote はなぜ分かれているのでしょうか？
There was a problem hiding this comment.
本番環境の動作を確認する必要があるかなあ、と考えたので、自分でsupabaseのクラウド環境にリモートのagentのプロジェクトを（カリキュラムで実施したように）作成しており、ターミナルでremoteとlocalを別々に起動できるように設定しています。よって、package.jsonにそのコマンドを設定しています。
なので、package.jsonは当初プルリクエストには含めるべきではないなあ、と考えていたのですが、今回含めたのは、この動作確認の必要性を確認したかったからです。ですので、駒形さんなどに確認し、無用！であれば、このpackage.jsonはmergeすべきではないので、自分の環境のみで使用したいと考えます。
また、想定remoteの方の環境変数は、当然、Git管理していませんので、（.gitignoreするようにしている）レビュアーには見ることができませんが、自分のPCのプロジェクト上では.env.remoteとenv.localとして別管理しています。
supabaseクラウド環境（リモートの Supabase）←本番環境（想定）で動作確認
NEXT_PUBLIC_SUPABASE_URL=https://省略
NEXT_PUBLIC_SUPABASE_ANON_KEY=・・・・
ローカル環境（self-hosted）←こちらでチーム開発
NEXT_PUBLIC_SUPABASE_URL=http://127.0.0.1:54321
NEXT_PUBLIC_SUPABASE_ANON_KEY=eyJhbGciOiJI...
上記の理由になります。混乱させてしまい、申し訳ありません。。。
There was a problem hiding this comment.
@supabase/postgrest-js と npm-run-all は必要ないと思うのですがいかがでしょうか？
There was a problem hiding this comment.
すいません。 npm-run-all は必要かもしれません。
|
お手隙の際にご確認をお願いします。🙏 今回は、以下の変更を行いました： 主な変更点
レビューのポイントと思われる点
|
…トと認証処理を修正 - `supabase.auth.getUser()` を `supabase.auth.getSession()` に変更 - `supabaseServer.ts` を async 関数に修正し、cookies の取得を適切に処理 - 環境変数の参照方法を `?? ""` に変更し、エラーを防止
Jyoumama
force-pushed
the
feature/create-companies-list
branch
from
5207021 to
7e6fcc0
Compare
…修正。 - `Header` コンポーネントのレンダリング方法を `await Header()` から `render(<Header />)` に変更。 ユーザー情報を非同期で取得するようになり、テストが確実に順番通りに実行されるようになりました。
- の呼び出しを非同期で行い、 を直接返さずに適切なユーザーオブジェクトを返すように修正。 これにより、モックユーザーが非同期に設定され、テストの実行が確実に同期的に行われます。
- をインポートし、プラグイン設定を修正。 - から に変更し、ESモジュール形式に対応。 Tailwind CSSのアニメーション機能を有効にし、設定方法をよりモジュール化しました。
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@daiiiichi 一からPRを作った方がやりやすいのであればそちらでもOKですよ〜 |
4 participants
Issue
概要
🔹 PRの目的
このプルリクエストでは、企業情報一覧ページの作成と、それに伴うコードの改善を行いました。
🔹 主な変更点
1.企業情報一覧ページの作成
fetchCompanies関数のエラーハンドリングを強化。database.types.tsを適用し、型定義を更新。pjord_svg.code-searchという作業途中で意図せず作成してしまった検索関連ファイルを削除。database.types.new.tsを削除し、正式な型定義をdatabase.types.tsに反映。🔹 変更の確認方法
1.
feature/create-companies-listブランチをローカルにチェックアウト：📌 実施した主なコミット一覧
✅ 企業情報一覧ページの作成
変更点:
Companies.tsx) のレイアウトを作成。✅ 不要な検索関連ファイルの削除
変更点:
pjord_svg.code-searchという未使用のファイルを削除。✅ Supabaseの型定義とクライアント設定を更新
変更点:
database.types.tsをnpx supabase gen typesで最新化。supabase.tsの設定を調整し、型エラーを防ぐ対応を実施。✅ 一時的な database.types.new.ts の削除
変更点:
database.types.tsへの適用後、一時ファイルを削除。変更前
変更後