-
Notifications
You must be signed in to change notification settings - Fork 78
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
feat: Support <Link />
in Future API
#509
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 99d4e92 The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
config/src/Config.ts
Outdated
import type { ConfigDefinition } from "./ConfigDefinition"; | ||
|
||
export type Config<T extends ActivityDefinition<string>> = { | ||
activities: T[]; | ||
transitionDuration: number; | ||
initialActivity?: () => T["name"]; | ||
}; | ||
export interface Config<T extends ActivityDefinition<string>> | ||
extends ConfigDefinition<T> {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ConfigDefinition과 Config를 분리합니다
<Link />
component<Link />
in Future API
59b8734
to
6303b62
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다~
하나 궁금한 점이 있는데 preload 기능은 어떻게 대체되는건가요?
기본 동작이 너무 복잡해보여서 제거했어요. 추후에 참고: |
9e92895
to
99d4e92
Compare
내용
<Link />
컴포넌트는@stackflow/plugin-history-sync
에 의존하고 있었어요. (import
를 함)<Link />
컴포넌트는@stackflow/plugin-history-sync
에 직접적으로 의존하지 않는 형태에요.@stackflow/plugin-history-sync
가 깔려있으면,href="..."
를 셋팅하고, 아니면href
를 없애면 돼요.app.decorate()
방식을 참고했어요.