Skip to content

v20260622.0 #422

Merged
s2yeons merged 21 commits into
mainfrom
develop
Jun 23, 2026
Merged

v20260622.0 #422
s2yeons merged 21 commits into
mainfrom
develop

Conversation

@s2yeons

@s2yeons s2yeons commented Jun 22, 2026

Copy link
Copy Markdown
Member

개요 💡

develop 브랜치의 내용을 main 브랜치에 반영합니다.

s2yeons and others added 21 commits June 15, 2026 14:16
…eanup

[chore] Vercel 이전 전에 불필요한 AWS Docker/CI-CD 설정 제거
API가 JSON이 아닌 HTML을 반환할 때 response.json()이 SyntaxError를 던지는 문제 수정.
상태 코드 확인 후 response.json()을 호출하도록 순서 변경.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
[chore] CI 워크플로우 추가
서버 Host 헤더(`headers().get('host')`)로 어드민 도메인을 판별하던 로직이
스테이지 리버스 프록시 환경에서 내부 호스트(localhost)를 받아 항상
localhost:3001로 리다이렉트되던 문제를 수정한다.

판별 기준을 브라우저 주소창 값인 window.location.origin으로 옮긴 클라이언트
컴포넌트 AdminRedirect로 교체했다. 서버 측 권한 검증(isAdminRole)은 그대로
유지된다.

resolves #418

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
[fix] 어드민 로그인 시 localhost:3001로 이동되는 문제 수정
…n-redirect

Revert "[fix] 어드민 로그인 시 localhost:3001로 이동되는 문제 수정"
…/admin-login-redirect

Revert "Revert "[fix] 어드민 로그인 시 localhost:3001로 이동되는 문제 수정""
…-419-fix/admin-login-redirect"

This reverts commit 319e3a6, reversing
changes made to 2199368.
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
hellogsm-client-26 Ready Ready Preview, Comment Jun 22, 2026 1:51pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
hellogsm-client-26-admin Skipped Skipped Jun 22, 2026 1:51pm

Request Review

@s2yeons s2yeons self-assigned this Jun 22, 2026

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request removes Dockerfiles, ignore files, and the standalone output configuration from Next.js configs. It also refactors the getOneseoList API function to check response status codes before parsing the JSON body, and adds onlyBuiltDependencies to the pnpm workspace configuration. The reviewer suggests using optional chaining (oneseoList?.data) when returning the API response data to prevent potential runtime errors if the response structure is unexpected.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines +47 to 49
const oneseoList = await response.json();

return oneseoList.data;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

API 응답 본문(oneseoList)이 비어있거나 예상치 못한 구조로 올 경우를 대비하여, 옵셔널 체이닝(?.)을 사용해 안전하게 data 프로퍼티에 접근하는 것을 권장합니다. 함수의 반환 타입이 Promise<OneseoListType | undefined>이므로 oneseoList?.data를 반환하면 런타임 에러를 방지하면서 안전하게 처리할 수 있습니다.

Suggested change
const oneseoList = await response.json();
return oneseoList.data;
const oneseoList = await response.json();
return oneseoList?.data;

@junjuny0227 junjuny0227 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@s2yeons s2yeons merged commit c551219 into main Jun 23, 2026
4 checks passed
@s2yeons s2yeons deleted the develop branch June 23, 2026 01:00
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.

3 participants