Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SystemVersion 동작 리팩토링 #431

Merged
merged 12 commits into from
Apr 26, 2024
Merged

SystemVersion 동작 리팩토링 #431

merged 12 commits into from
Apr 26, 2024

Conversation

XiNiHa
Copy link
Contributor

@XiNiHa XiNiHa commented Apr 24, 2024

SystemVersion의 동작을 리팩토링합니다.

  • 이제 릴리즈 노트, 기술블로그 등의 URL에 ?v=가 붙지 않습니다.
  • 서버에서 SystemVersion을 가져올 때 prop drilling 대신 AsyncLocalStorage를 사용합니다.
  • 각 문서가 대상으로 하는 모듈 버전을 지정할 수 있도록 합니다.
    • 단일 버전을 대상으로 하는 문서의 경우, URL 내 버전 파라미터를 해당 버전으로 덮어씌웁니다.
      (예: /docs/ko/v2-payment/v2?v=v1 접속 시에도 /docs/ko/v2-payment/v2?v=v2로 덮어씌워짐)
    • 타 버전에 대응되는 내용의 문서가 있을 경우, 버전 토글 클릭 시 해당 페이지로 이동하도록 수정합니다.
  • 버전토글 클릭 시 페이지 내용이 변하는 경우 (VersionGate가 사용되었든, 타 버전에 대응되는 문서가 있든) 버전토글에 5초간 팝오버를 띄웁니다.

Closes #346

@XiNiHa XiNiHa requested a review from CirnoV April 24, 2024 07:49
Copy link

vercel bot commented Apr 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developers ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2024 2:38pm

package.json Show resolved Hide resolved
src/middleware.ts Outdated Show resolved Hide resolved
const docCollection = defineCollection({
schema: z.object({
title: z.string(),
description: z.string(),
targetVersions: z.array(SystemVersion).optional(),
versionVariants: z.record(SystemVersion, z.string()).optional(),
Copy link
Member

@CirnoV CirnoV Apr 25, 2024

Choose a reason for hiding this comment

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

이것도 #30 하는김에 같이 해결해야겠네요.
AST Explorer에서 보니 remark-frontmatter 플러그인이 적용된 상태에서는 yaml 노드로 파싱되더라구요. 이러면 전에 만든 lint 플러그인만 수정해줘도 될 것 같아요.

@sso-ashley sso-ashley merged commit 1eb16b4 into main Apr 26, 2024
3 checks passed
@sso-ashley sso-ashley deleted the fix/issue-346 branch April 26, 2024 07:37
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