Conversation
There was a problem hiding this comment.
Pull request overview
This PR prepares the repo for the v1.1.0 release by aligning versions, updating documentation/policies, improving subscription hook robustness around auth/App Check retry semantics, and fixing a stale memo dependency in BoardPage.
Changes:
- Added/updated release documentation (
CHANGELOG.md, refreshedREADME.md, updatedSECURITY.md). - Bumped frontend + functions versions/locks to
1.1.0. - Refactored
useBoards/useIncomingInvitesto key loading/subscription state byuid/emailand adjustedBoardPagememo dependencies.
Reviewed changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/pages/BoardPage.jsx | Fixes useMemo dependency array so parent-candidate lists recompute when board type flags change. |
| src/hooks/useIncomingInvites.js | Refactors invites subscription state keyed by email and derives loading from forEmail. |
| src/hooks/useBoards.js | Refactors boards subscription state keyed by uid, integrates App Check retry state, derives loading from forUid. |
| SECURITY.md | Updates supported-version policy and private vulnerability reporting guidance. |
| README.md | Refreshes feature/set-up/deploy documentation for v1.1.0 and the board hierarchy/membership model. |
| package.json | Bumps frontend version to 1.1.0. |
| package-lock.json | Updates lockfile metadata to 1.1.0 (and associated lock updates). |
| functions/package.json | Bumps functions package version to 1.1.0. |
| functions/package-lock.json | Updates functions lockfile metadata to 1.1.0. |
| CHANGELOG.md | Adds changelog with 1.1.0 notes and prior release entries. |
Files not reviewed (1)
- functions/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d474331e27
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 24cd7566be
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 24cd7566be
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Fix stale board/invite hook state across auth transitions
Motivation
v1.1.0release with focused features like Excel export and super-board support and document notable changes.functionspackage and refresh documentation and security guidance.Description
CHANGELOG.mddocumenting thev1.1.0release and prior notes.README.mdto describe features (Excel export, super boards, invites, membership model), setup, deployment, and release status.SECURITY.mdto a supported-version table and private vulnerability reporting guidance.1.1.0inpackage.jsonandfunctions/package.jsonand update correspondingpackage-lock.jsonentries.useBoardsto key subscriptions byuid, centralize subscription state into a singlestateobject, computeloadingfrom aforUidmarker, and return early when unauthenticated to avoid unnecessary subscriptions.useIncomingInvitessimilarly to key onemail, hold state in a single object withforEmailfor accurateloadingsemantics, and return early when unauthenticated.BoardPagememo dependencies by addingisSuperBoardandisSubBoardto theuseMemodependency array to ensureparentCandidatesrecalculates correctly when board type changes.Testing
Codex Task