Skip to content

팀 이름 수정 UI 연결 - #117

Merged
hwangjokim merged 2 commits into
mainfrom
feat/116-team-name-editing-ui
Jun 17, 2026
Merged

팀 이름 수정 UI 연결#117
hwangjokim merged 2 commits into
mainfrom
feat/116-team-name-editing-ui

Conversation

@hwangjokim

Copy link
Copy Markdown
Member

Summary

  • 팀 관리 탭에서 방장이 팀 이름을 수정하고 저장할 수 있게 연결했습니다.
  • ProjectGroups API 클라이언트, TanStack Query mutation, MSW mock을 추가했습니다.
  • OpenAPI와 팀 스페이스 API 메모에 팀 이름 수정 계약을 반영했습니다.

Validation

  • pnpm tsc --noEmit: pass
  • pnpm biome lint .: pass
  • pnpm build: pass

Closes #116

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
client Ready Ready Preview, Comment Jun 17, 2026 1:20am

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 570e2c3fda

ℹ️ 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".

Comment thread src/lib/api/project-groups.ts Outdated
projectName,
}: UpdateProjectGroupNameRequest) {
return apiRequest<void>(`/project-groups/${projectGroupId}/name`, {
body: JSON.stringify({ projectName }),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use the JSON request option for name updates

When this runs against the real backend, this raw body path does not get the Content-Type: application/json header because apiRequest only adds that header for the json option. The new OpenAPI endpoint declares an application/json request body, so servers that rely on the media type for JSON parsing/content negotiation can reject or ignore the project name update even though the MSW handler accepts request.json(). Use json: { projectName } here like the other API calls.

Useful? React with 👍 / 👎.

@hwangjokim
hwangjokim merged commit 16cba61 into main Jun 17, 2026
3 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.

팀 이름 수정 UI 연결

1 participant