Skip to content

fix:hookの呼び出し順序の修正 #60#61

Merged
ponmalnya merged 1 commit intomasterfrom
fixRulesofhooks#60
Mar 11, 2026
Merged

fix:hookの呼び出し順序の修正 #60#61
ponmalnya merged 1 commit intomasterfrom
fixRulesofhooks#60

Conversation

@ponmalnya
Copy link
Copy Markdown
Collaborator

No description provided.

@ponmalnya ponmalnya requested a review from Copilot March 11, 2026 02:36
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
aima Ready Ready Preview, Comment Mar 11, 2026 2:36am

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

ReviewClient で React Hooks の呼び出し順序が条件分岐で崩れていた点を修正し、Hooks ルール違反を解消するPRです(#60)。

Changes:

  • useState などの Hooks 呼び出しを条件付き return より前に移動
  • クエリパラメータのパース処理位置を整理(Hooks 呼び出し順序が安定するように調整)

@ponmalnya ponmalnya marked this pull request as ready for review March 11, 2026 02:41
@ponmalnya ponmalnya merged commit d6e37d0 into master Mar 11, 2026
8 of 9 checks passed
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