Skip to content
This repository was archived by the owner on Nov 26, 2025. It is now read-only.

feat: update gtm link url#209

Merged
pitzcarraldo merged 2 commits into
mainfrom
gtm-link
Nov 12, 2025
Merged

feat: update gtm link url#209
pitzcarraldo merged 2 commits into
mainfrom
gtm-link

Conversation

@pitzcarraldo
Copy link
Copy Markdown
Contributor

@pitzcarraldo pitzcarraldo commented Nov 12, 2025

Summary

  • update gtm link url

Summary by CodeRabbit

  • 문서

    • 개발자 가이드(GTM) 및 사용자 가이드(Cafe24 GTM)에서 Notifly GTM 템플릿의 다운로드 링크를 최신 저장소로 갱신하여 최신 리소스 접근성을 향상시켰습니다.
    • 사이트 내비게이션 및 푸터의 Notifly 홈페이지 링크 설명을 www 없이 표준 도메인(notifly.tech)으로 업데이트했습니다.
  • 잡무(Chores)

    • 지속적 통합(CI) 워크플로우를 추가하여 빌드 검증 절차를 자동화했습니다.

@pitzcarraldo pitzcarraldo self-assigned this Nov 12, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Nov 12, 2025

Walkthrough

한국어 문서 두 건의 Notifly GTM 템플릿 다운로드 URL을 team-michael에서 notifly-tech로 변경하고, 사이트 구성 링크(https://www.notifly.techhttps://notifly.tech) 업데이트 및 GitHub Actions CI 워크플로우 추가를 포함한 여러 구성/문서 변경을 적용했습니다.

Changes

집단 / 파일 변경 요약
문서: GTM 템플릿 링크 업데이트
i18n/ko/docusaurus-plugin-content-docs/current/developer-guide/client-sdk/google-tag-manager.mdx, i18n/ko/docusaurus-plugin-content-docs/current/user-guide/cafe24/cafe24-gtm.mdx
Notifly GTM 템플릿 다운로드 URL을 team-michael 저장소에서 notifly-tech 저장소로 변경
구성: 사이트 URL 업데이트
docusaurus.config.js, i18n/ko/docusaurus-theme-classic/footer.json
사이트 링크 및 푸터 설명의 도메인에서 https://www.notifly.techhttps://notifly.tech로 변경
CI: GitHub Actions 추가
.github/workflows/ci.yml
CI 워크플로우 추가: 푸시(PR 제외 main 제외) 및 PR(main 대상) 트리거, Node.js 24 환경에서 checkout → setup-node → npm ci → npm run build 실행 및 캐시 설정 등

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10분

  • 문서 링크와 구성 URL 변경은 단순 검증(링크 대상, 오타)으로 충분
  • CI 워크플로우 추가: 실행 조건, Node 버전, 캐시 키(package-lock.json 기반) 및 빌드 스텝 확인 필요
  • 주의할 파일:
    • .github/workflows/ci.yml (워크플로우 트리거/보안/캐시 동작)
    • docusaurus.config.jsi18n/ko/.../footer.json (링크 정확성)
    • 변경된 MDX 파일들(외부 링크 대상 확인)

Suggested reviewers

  • minyonglee

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명은 필수 섹션(Summary, Describe your changes, Issue ticket number and link) 중 Summary만 포함하고 있으며, 나머지 섹션이 완전히 누락되어 있습니다. PR 설명을 완성하세요. 'Describe your changes' 섹션에 변경 사항의 상세 설명과 'Issue ticket number and link' 섹션을 추가해야 합니다.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 'feat: update gtm link url'은 변경 사항의 핵심을 명확하게 요약하고 있습니다. GTM 링크 URL 업데이트라는 주요 변경 내용을 정확하게 나타냅니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch gtm-link

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 346ca79 and 031e15b.

📒 Files selected for processing (3)
  • .github/workflows/ci.yml (1 hunks)
  • docusaurus.config.js (2 hunks)
  • i18n/ko/docusaurus-theme-classic/footer.json (1 hunks)
🧰 Additional context used
🪛 actionlint (1.7.8)
.github/workflows/ci.yml

15-15: the runner of "actions/checkout@v3" action is too old to run on GitHub Actions. update the action's version to fix this issue

(action)

🔇 Additional comments (3)
docusaurus.config.js (1)

136-136: URL 업데이트가 정확합니다.

네비게이션 바와 푸터에서 URL을 https://www.notifly.tech에서 https://notifly.tech로 일관되게 업데이트했습니다.

Also applies to: 159-159

i18n/ko/docusaurus-theme-classic/footer.json (1)

16-16: 메타데이터 업데이트가 올바릅니다.

푸터 링크 설명이 실제 URL 변경사항과 일치하도록 정확하게 업데이트되었습니다.

.github/workflows/ci.yml (1)

18-18: 검토가 필요 없습니다. Node.js 버전 24는 올바릅니다.

Node.js v24.x는 2025년 11월 기준 Active LTS 버전이므로 프로덕션 환경에 적합합니다. 현재 CI 구성이 올바르며, 변경이 필요하지 않습니다.

Comment thread .github/workflows/ci.yml
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

오래된 액션 버전을 업데이트하세요.

actions/checkout@v3는 GitHub Actions에서 실행하기에는 너무 오래된 버전입니다. 최신 버전인 v4로 업데이트해야 합니다.

다음 diff를 적용하여 수정하세요:

-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- uses: actions/checkout@v3
- uses: actions/checkout@v4
🧰 Tools
🪛 actionlint (1.7.8)

15-15: the runner of "actions/checkout@v3" action is too old to run on GitHub Actions. update the action's version to fix this issue

(action)

🤖 Prompt for AI Agents
.github/workflows/ci.yml around line 15: the workflow uses an outdated
actions/checkout@v3; update the action reference to actions/checkout@v4 to use
the latest stable version, and ensure no other steps depend on v3-specific
behavior so the workflow continues to run correctly.

@pitzcarraldo pitzcarraldo merged commit 1b928bf into main Nov 12, 2025
4 checks passed
@pitzcarraldo pitzcarraldo deleted the gtm-link branch November 12, 2025 11:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant