Conversation
dev to main
dev to main
fix: 딥링크 접근 권한 허용 [To Main]
dev to main
fix: ask 전용 딥링크 파일 cloudflare 배포 설정 변경
fix: 안드로이드 assetlinks 파일 내용 수정
| content: `${activity.generation}기 ${activity.part}`, | ||
| content: `${activity.generation}기 ${activity.team || activity.part}`, | ||
| isActive: activity.generation === LATEST_GENERATION, |
There was a problem hiding this comment.
{
"id": 1898,
"generation": 35,
"part": "웹",
"team": "미디어팀"
}미디어팀일 경우 미디어팀으로 표기될 것 같아요!
Summary by CodeRabbit
Walkthrough.env.development의 인증 API URL 변경, GitHub Actions 빌드에서 배포 패키지 생성 단계 추가, .well-known 경로에 Apple/Android 매니페스트를 서빙하는 Pages 함수 추가, API 401 처리에 well-known 경로 토큰 갱신 우회 로직 추가, 멤버 리스트 배지 텍스트 로직 수정. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts (beta)
No actionable comments were generated in the recent review. 🎉 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/components/members/main/MemberList/index.tsx (1)
491-497:⚠️ Potential issue | 🟡 Minor
Profile.activities타입 정의 오류 수정 필요
Profile타입에서activities[].team이string으로 정의되어 있으나, 실제 데이터는null또는 빈 문자열("")을 포함할 수 있습니다.src/pages/members/edit.tsx,src/pages/members/upload.tsx등에서activity.team === UNSELECTED || activity.team === ''체크를 통해 빈 문자열을 명시적으로null로 변환하는 패턴이 있으므로,team: string | null으로 수정하세요.현재 코드의
||연산자는null과 빈 문자열 모두를 처리하므로 올바른 선택입니다. 따라서 연산자를??로 변경할 필요는 없습니다.
🤖 Fix all issues with AI agents
In @.env.development:
- Line 14: The NEXT_PUBLIC_CREW_API_URL environment variable currently includes
a trailing slash which can cause double-slash URLs when concatenated with
endpoints (e.g., axios baseURL + "/internal/post/{id}"); remove the trailing
slash from the NEXT_PUBLIC_CREW_API_URL value so it matches the other URL vars
(e.g., change to https://crew-api-dev.sopt.org) and also ensure the
.env.development file ends with a newline character; update the value for
NEXT_PUBLIC_CREW_API_URL and save the file with a final newline.
🧹 Nitpick comments (3)
.github/workflows/build.yml (1)
44-48:functions디렉토리 부재 시 빌드 실패 가능성
cp -r functions deploy/명령은functions디렉토리가 존재하지 않으면 빌드가 실패합니다. 다른 브랜치나 향후 해당 디렉토리가 제거될 경우를 대비해 방어적으로 처리하는 것이 좋습니다.제안
run: | mkdir -p deploy cp -r out/* deploy/ - cp -r functions deploy/ + if [ -d functions ]; then cp -r functions deploy/; fipublic/.well-known/apple-app-site-association (1)
1-13: 정적 파일과 서버리스 함수 중복 제공
public/.well-known/apple-app-site-association(정적 파일)과functions/.well-known/apple-app-site-association.ts(서버리스 함수)가 동일한 콘텐츠를 제공합니다. Cloudflare Pages에서functions/라우트가 정적 파일보다 우선하므로 이 정적 파일은 실제로 서빙되지 않을 수 있습니다. 둘 중 하나만 유지하거나, 정적 파일이 폴백 용도인지 명확히 하는 것이 좋습니다.functions/.well-known/assetlinks.json.ts (1)
1-13: JSON 템플릿 리터럴 내 들여쓰기 불일치
sha256_cert_fingerprints배열 항목(Line 8-9)의 들여쓰기가 나머지 JSON 구조와 일치하지 않습니다. 기능적 문제는 없지만 가독성을 위해 정렬하는 것이 좋습니다.제안
const ASSETLINKS_JSON = `[ { "relation": ["delegate_permission/common.handle_all_urls"], "target": { "namespace": "android_app", "package_name": "org.sopt.official", "sha256_cert_fingerprints": [ - "29:23:1F:E3:7A:FD:76:9C:58:CA:F3:E2:D9:7C:A1:69:CF:04:A8:6E:5B:8A:C6:56:6A:6F:E8:FD:D8:FF:47:43", - "30:8A:35:1F:E1:88:94:6F:BC:E9:B2:D5:FC:2F:9B:F2:10:35:2A:43:B8:36:03:8B:C4:1B:EA:51:23:76:1C:00" + "29:23:1F:E3:7A:FD:76:9C:58:CA:F3:E2:D9:7C:A1:69:CF:04:A8:6E:5B:8A:C6:56:6A:6F:E8:FD:D8:FF:47:43", + "30:8A:35:1F:E1:88:94:6F:BC:E9:B2:D5:FC:2F:9B:F2:10:35:2A:43:B8:36:03:8B:C4:1B:EA:51:23:76:1C:00" ] } } ]`;
|
🚀 프리뷰 배포 확인하기 🚀 |
4 participants
🤫 쉿, 나한테만 말해줘요. 이슈넘버
🧐 어떤 것을 변경했어요~?
🤔 그렇다면, 어떻게 구현했어요~?
❤️🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.
📸 스크린샷, 없으면 이것 참,, 섭섭한데요?