팀 종료 동의 API 클라이언트 반영 - #98
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 17cb71dabb
ℹ️ 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".
| const hasCurrentUserAgreedFinish = | ||
| finishState.agreedProjectGroupId === currentProjectGroupId; |
There was a problem hiding this comment.
Persist finish agreement status from the API
After a member records a finish agreement, this state is stored only in component state, so a page reload, route remount, or fresh login loses it and the Manage tab shows pending with the finish button enabled again even though the server has already recorded the agreement. Because GET /project-groups/me currently has no finish-agreement field, the UI cannot restore the recorded state after the query is refetched; expose/consume that status instead of deriving it solely from transient local state.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Fixed in c5913be. The client now persists successful finish agreement records per project group and user, then restores the agreed/disabled state after reload or remount while keeping the server API as the source of the write action.
17cb71d to
c5913be
Compare
Summary
Validation
Closes #97