docs: sync Navigating Activities section (KR/EN) #622
Conversation
|
📝 WalkthroughSummary by CodeRabbit
WalkthroughReplaces “stack” with “activity” in English and Korean docs; clarifies pop() first parameter can be a count or an options object (and is optional); adds explicit usage examples; updates code fence attributes and Article.tsx example handlers/buttons to use goBack() and goBackMultiple() invoking pop() (including pop(3)). Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Article as Article.tsx
participant Nav as navigation API
User->>Article: Click "Back" button
Article->>Nav: pop() %% single activity
User->>Article: Click "Back 3 Steps"
Article->>Nav: pop(3) %% pop multiple activities
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15 minutes Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
Status, Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/pages/docs/get-started/navigating-activities.ko.mdx (1)
191-193: Thanks for incorporating the previous suggestionThe explanation about using the first parameter as count or options, and the second as options when the first is a count, matches the prior feedback.
🧹 Nitpick comments (7)
docs/pages/docs/get-started/navigating-activities.ko.mdx (7)
138-138: Verify code-block meta: is "/pop/" supported by your MDX/Nextra setup?"copy" is familiar, "showLineNumbers" looks fine, but the trailing "/pop/" token is non-standard in many setups. If this isn't intentionally handled by your code-block plugin (e.g., to highlight matches), it may render as plain text or be ignored.
If unsupported, drop it:
-```tsx showLineNumbers filename="Article.tsx" copy /pop/ +```tsx showLineNumbers filename="Article.tsx" copy
155-159: Clarify behavior when popping more steps than availableIf the stack has fewer than 3 entries, what happens? Do you clamp to the root, throw, or no-op? Consider adding a one-liner note to set user expectations.
Want me to draft a short Callout explaining the edge cases (e.g., count <= 0, count > stack size)?
174-176: Korean spacing nit: '첫번째' → '첫 번째'Apply standard spacing for ordinal numbers.
-첫번째 파라미터는 생략 가능하며, 생략 시 기본값이 사용돼요. +첫 번째 파라미터는 생략 가능하며, 생략 시 기본값이 사용돼요.
182-185: Show a concrete options example to reduce ambiguityConsider demonstrating a real option (e.g., animate) so readers see the shape.
-pop({ - /* 추가 옵션 */ -}); // 추가 옵션과 함께 스택 하나 제거 +pop({ + animate: false, +}); // 애니메이션 없이 스택 하나 제거
186-189: Mirror the concrete options example for the count+options overloadKeeps both overload examples symmetric and clearer.
-pop(3, { - /* 추가 옵션 */ -}); // 추가 옵션과 함께 스택 여러 개 제거 +pop(3, { + animate: false, +}); // 애니메이션 없이 스택 여러 개 제거
191-193: Korean spacing nit: '첫번째/두번째' → '첫 번째/두 번째'Apply standard spacing consistently.
-`pop()` 함수의 첫번째 파라미터는 제거할 스택의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. -첫번째 파라미터를 스택 개수로 사용하면, 두번째 매개변수는 추가 옵션을 전달하는 데 사용할 수 있어요. +`pop()` 함수의 첫 번째 파라미터는 제거할 스택의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. +첫 번째 파라미터를 스택 개수로 사용하면, 두 번째 매개변수는 추가 옵션을 전달하는 데 사용할 수 있어요.
191-195: Minor wording consistency with following sentenceRight after this block, the sentence says "첫번째 파라미터인 추가 옵션" which can be misleading since the first parameter can be either count or options. Consider rephrasing to avoid implying it is always options.
Proposed rephrase (for Line 194, for consistency with Lines 191–193):
- "
pop()함수의 옵션에는 다음과 같은 값이 있어요."- Or: "
pop()함수에 전달할 옵션 객체에는 다음과 같은 값이 있어요."
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/pages/docs/get-started/navigating-activities.ko.mdx(3 hunks)
🔇 Additional comments (4)
docs/pages/docs/get-started/navigating-activities.ko.mdx (4)
150-153: LGTM: Simple back handler showcases default pop()Clear and accurate example of removing a single activity with the default behavior.
164-166: LGTM: Handlers and labels are consistent with the exampleButtons are wired correctly to their respective handlers; example reads well.
178-178: LGTM: pop() default example is clearConcise and correct.
180-181: LGTM: Multiple-pop example is correctGood illustration of removing multiple activities.
ENvironmentSet
left a comment
ENvironmentSet
left a comment
There was a problem hiding this comment.
기여 감사드립니다. 정정해야 할 용어 사용이 있어서 요청드렸어요.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/pages/docs/get-started/navigating-activities.ko.mdx (1)
191-194: Disambiguate parameter roles and fix terminology/consistency
- 첫 문장은 정확하지만, 바로 아래 문단의 “첫번째 파라미터인 추가 옵션”은 오해를 부릅니다. 첫 번째 파라미터는 “개수 또는 옵션”이므로, “옵션에는 …”으로 표현하는 게 명확합니다.
- ‘첫번째/두번째’ → ‘첫 번째/두 번째’ 띄어쓰기 교정.
- ‘매개변수’와 ‘파라미터’를 혼용하고 있어요. 본 문서 전체에서 이미 ‘파라미터’를 주로 쓰고 있으니 이에 맞춰 통일을 제안합니다.
-`pop()` 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. -첫번째 파라미터를 액티비티 개수로 사용하면, 두번째 매개변수는 추가 옵션을 전달하는 데 사용할 수 있어요. - -`pop()` 함수의 첫번째 파라미터인 추가 옵션에는 다음과 같은 값이 있어요. +`pop()` 함수의 첫 번째 파라미터는 제거할 액티비티 개수(number) 또는 추가 옵션(object)을 받을 수 있어요. +첫 번째 파라미터를 개수로 사용하는 경우, 두 번째 파라미터로 옵션을 전달할 수 있어요. + +`pop()` 함수의 옵션에는 다음과 같은 값이 있어요.추가로, 위와 같은 띄어쓰기/용어 통일을 문서 상단의 ‘push()/replace()’ 설명부(예: 첫번째/두번째)에도 일괄 적용하면 더 깔끔합니다. 필요하시면 일괄 패치 제안 드리겠습니다.
🧹 Nitpick comments (5)
docs/pages/docs/get-started/navigating-activities.ko.mdx (5)
78-78: Wording tweak for natural Korean (avoid awkward '액티비티에 ... 추가')“액티비티에 새로운 액티비티를 추가하지 않고”는 어색합니다. “새로운 액티비티를 추가하지 않고”로 간결하게 다듬는 것을 제안해요.
-다음으로 액티비티에 새로운 액티비티를 추가하지 않고 현재 액티비티를 교체하는 방법에 대해서 살펴봐요. `stackflow.ts`에서 생성했던 `useFlow()` 훅의 `replace()` 함수를 통해 다음과 같이 현재 액티비티를 교체할 수 있어요. +다음으로 새로운 액티비티를 추가하지 않고 현재 액티비티를 교체하는 방법에 대해서 살펴봐요. `stackflow.ts`에서 생성했던 `useFlow()` 훅의 `replace()` 함수를 통해 다음과 같이 현재 액티비티를 교체할 수 있어요.
150-159: Optional: include an example with options (animate: false) for completenessYou already demonstrate count-based pop below; showing a quick options-based call here would complete the story.
const goBack = () => { // 액티비티 하나만 제거 - pop(); + pop(); }; const goBackMultiple = () => { // 액티비티 여러 개 제거 pop(3); }; + + // 예: 애니메이션 없이 하나만 제거 + // const goBackWithoutAnimation = () => pop({ animate: false });
165-166: Optional: localize button labels to Korean for consistency본문이 한국어인 만큼 버튼 라벨도 한국어로 맞추면 일관성이 높아집니다.
- <button onClick={goBack}>Back</button> - <button onClick={goBackMultiple}>Back 3 Steps</button> + <button onClick={goBack}>뒤로 가기</button> + <button onClick={goBackMultiple}>3단계 뒤로</button>
174-176: Clarify default behavior and spacing of ordinal numbers기본값을 구체적으로 명시하면 이해가 쉬워집니다. 또한 ‘첫번째’ → ‘첫 번째’로 띄어쓰기 교정 제안합니다.
-`pop()` 함수는 제거할 액티비티 개수와 추가 옵션을 위한 선택적 파라미터를 받을 수 있어요. -첫번째 파라미터는 생략 가능하며, 생략 시 기본값이 사용돼요. +`pop()` 함수는 제거할 액티비티 개수와 추가 옵션을 위한 선택적 파라미터를 받을 수 있어요. +첫 번째 파라미터는 생략 가능하며, 생략 시 기본값(1개)이 사용돼요.
204-204: Fix typo‘액티비티을’ → ‘액티비티를’
-액티비티를 쌓고, 교체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 액티비티을 만들 수 있는 방법에 대해 알아봐요. +액티비티를 쌓고, 교체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 액티비티를 만들 수 있는 방법에 대해 알아봐요.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
docs/pages/docs/get-started/navigating-activities.ko.mdx(5 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/pages/docs/get-started/navigating-activities.ko.mdx
[grammar] ~191-~191: Ensure spelling is correct
Context: ...추가 옵션과 함께 액티비티 여러 개 제거 ``` pop() 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[grammar] ~204-~204: Ensure spelling is correct
Context: ...체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 액티비티을 만들 수 있는 방법에 대해 알아봐요.
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🔇 Additional comments (2)
docs/pages/docs/get-started/navigating-activities.ko.mdx (2)
138-172: Nice example improvements and code fence metadatashowLineNumbers/filename/copy and the focused pop example look good. The Article component wiring is correct.
178-189: Examples look good다양한 호출 형태가 명확하게 제시되어 있어요. 독자가 빠르게 패턴을 파악할 수 있습니다.
SEOKKAMONI
force-pushed
the
docs/sync-navigating-activities
branch
from
2dc0f52 to
afe3745
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/pages/docs/get-started/navigating-activities.ko.mdx (1)
174-189: 과거 리뷰 피드백 반영 확인: ‘스택’ → ‘액티비티’.해당 구간 전반에서 ‘스택’ 용어가 ‘액티비티’로 일관되게 교정되었습니다. 감사합니다.
🧹 Nitpick comments (8)
docs/pages/docs/get-started/navigating-activities.en.mdx (2)
179-186: Pluralize “additional option” in inline comments for consistency.The object holds options; pluralize to match surrounding text (“additional options”).
Apply this diff:
pop({ - /* additional option */ + /* additional options */ }); // pop a single activity with additional options pop(3, { - /* additional option */ + /* additional options */ }); // pop multiple activities with additional options
200-200: Fix grammar: “how to activity” → “how to stack.”Minor wording issue in the closing sentence.
Apply this diff:
-We have learned how to activity, replace, and delete activities. Now, let's learn how to create a virtual activity within an activity. +We have learned how to stack, replace, and delete activities. Now, let's learn how to create a virtual activity within an activity.docs/pages/docs/get-started/navigating-activities.ko.mdx (6)
78-78: 자연스러운 표현으로 다듬기 (“액티비티에” 삭제).문장 흐름상 ‘액티비티에’는 군더더기예요.
적용 diff:
-다음으로 액티비티에 새로운 액티비티를 추가하지 않고 현재 액티비티를 교체하는 방법에 대해서 살펴봐요. `stackflow.ts`에서 생성했던 `useFlow()` 훅의 `replace()` 함수를 통해 다음과 같이 현재 액티비티를 교체할 수 있어요. +다음으로 새로운 액티비티를 추가하지 않고 현재 액티비티를 교체하는 방법을 살펴봐요. `stackflow.ts`에서 생성했던 `useFlow()` 훅의 `replace()` 함수를 통해 다음과 같이 현재 액티비티를 교체할 수 있어요.
150-158: 예제 보강(LGTM) + 버튼 레이블 현지화 제안.goBackMultiple 예제 추가는 유익합니다. 버튼 레이블을 한국어로 현지화하면 일관성이 더 좋아져요.
예시:
- <button onClick={goBack}>Back</button> - <button onClick={goBackMultiple}>Back 3 Steps</button> + <button onClick={goBack}>뒤로 가기</button> + <button onClick={goBackMultiple}>3단계 뒤로 가기</button>
174-176: 맞춤법: 띄어쓰기(‘첫 번째’).국립국어원 표기 기준에 따라 ‘첫번째’ → ‘첫 번째’로 고쳐주세요.
적용 diff:
-`pop()` 함수는 제거할 액티비티 개수와 추가 옵션을 위한 선택적 파라미터를 받을 수 있어요. -첫번째 파라미터는 생략 가능하며, 생략 시 기본값이 사용돼요. +`pop()` 함수는 제거할 액티비티 개수와 추가 옵션을 위한 선택적 파라미터를 받을 수 있어요. +첫 번째 파라미터는 생략 가능하며, 생략 시 기본값이 사용돼요.
191-193: 용어 일관성과 띄어쓰기 정리(‘파라미터’/‘첫 번째’/‘두 번째’).앞 문단에서 ‘파라미터’를 사용하므로 동일 용어로 통일하고, 띄어쓰기 교정해요.
적용 diff:
-`pop()` 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. -첫번째 파라미터를 액티비티 개수로 사용하면, 두번째 매개변수는 추가 옵션을 전달하는 데 사용할 수 있어요. +`pop()` 함수의 첫 번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. +첫 번째 파라미터를 액티비티 개수로 사용하면, 두 번째 파라미터로 추가 옵션을 전달할 수 있어요.
194-200: 문장 정확도: “첫번째 파라미터인 추가 옵션” 표현 수정.첫 번째 파라미터가 수 또는 옵션을 모두 받을 수 있어 혼동의 소지가 있습니다. 추가 옵션 설명으로 일반화해주세요.
적용 diff:
-`pop()` 함수의 첫번째 파라미터인 추가 옵션에는 다음과 같은 값이 있어요. +`pop()`의 추가 옵션에는 다음과 같은 값이 있어요.
204-204: 오탈자: ‘액티비티을’ → ‘액티비티를’.조사 교정이 필요합니다.
적용 diff:
-액티비티를 쌓고, 교체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 액티비티을 만들 수 있는 방법에 대해 알아봐요. +액티비티를 쌓고, 교체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 액티비티를 만들 수 있는 방법에 대해 알아봐요.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
docs/pages/docs/get-started/navigating-activities.en.mdx(2 hunks)docs/pages/docs/get-started/navigating-activities.ko.mdx(5 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/pages/docs/get-started/navigating-activities.ko.mdx
[grammar] ~191-~191: Ensure spelling is correct
Context: ...추가 옵션과 함께 액티비티 여러 개 제거 ``` pop() 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[grammar] ~204-~204: Ensure spelling is correct
Context: ...체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 액티비티을 만들 수 있는 방법에 대해 알아봐요.
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🔇 Additional comments (3)
docs/pages/docs/get-started/navigating-activities.en.mdx (2)
172-172: API note reads well; aligns with updated semantics.The clarification that both the count and options are optional is clear and consistent with examples below.
188-188: LGTM: pop() first-parameter overload explanation is clear.Nicely clarifies number vs. options in the first parameter and how the second is used when the first is numeric.
docs/pages/docs/get-started/navigating-activities.ko.mdx (1)
138-138: 코드 펜스 옵션 추가(LGTM).showLineNumbers/filename 옵션 추가로 예제 가독성이 좋아졌어요.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/pages/docs/get-started/navigating-activities.ko.mdx (3)
164-165: Localize button labels to Korean for consistencyThe doc is in Korean but the buttons are English. Consider localizing.
Apply:
- <button onClick={goBack}>Back</button> - <button onClick={goBackMultiple}>Back 3 Steps</button> + <button onClick={goBack}>뒤로</button> + <button onClick={goBackMultiple}>3단계 뒤로</button>
174-176: Polish Korean spacing and clarify the default behavior explicitly
- “첫번째” → “첫 번째” (띄어쓰기)
- Make the default explicit: popping 1 activity.
-`pop()` 함수는 제거할 액티비티 개수와 추가 옵션을 위한 선택적 파라미터를 받을 수 있어요. -첫번째 파라미터는 생략 가능하며, 생략 시 기본값이 사용돼요. +`pop()` 함수는 제거할 액티비티 개수와 추가 옵션을 위한 선택적 파라미터를 받을 수 있어요. +첫 번째 파라미터는 생략 가능하며, 생략 시 액티비티 1개가 제거돼요.
191-196: Clarify parameter semantics and fix phrasing (“첫 번째/두 번째”)
- “첫번째/두번째” → “첫 번째/두 번째” (띄어쓰기)
- The sentence “첫번째 파라미터는 … 추가 옵션을 정의할 수 있어요” is ambiguous because the first parameter can be either a number or the options object. Suggest being explicit.
- The line “첫번째 파라미터인 추가 옵션에는 …” is incorrect after your earlier clarification. It should describe the “추가 옵션 객체” itself, not tie it to the first parameter.
-`pop()` 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. -첫번째 파라미터를 액티비티 개수로 사용하면, 두번째 매개변수는 추가 옵션을 전달하는 데 사용할 수 있어요. +`pop()` 함수의 첫 번째 파라미터에는 제거할 액티비티 개수(숫자) 또는 옵션 객체를 전달할 수 있어요. +첫 번째 파라미터가 숫자일 때는 두 번째 매개변수로 옵션 객체를 전달할 수 있어요. @@ -`pop()` 함수의 첫번째 파라미터인 추가 옵션에는 다음과 같은 값이 있어요. +추가 옵션 객체에는 다음과 같은 값이 있어요.Also consider sweeping the doc for “첫번째/두번째/세번째” and applying correct spacing (“첫 번째/두 번째/세 번째”) for consistency.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
docs/pages/docs/get-started/navigating-activities.en.mdx(1 hunks)docs/pages/docs/get-started/navigating-activities.ko.mdx(3 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/pages/docs/get-started/navigating-activities.en.mdx
🧰 Additional context used
🪛 LanguageTool
docs/pages/docs/get-started/navigating-activities.ko.mdx
[grammar] ~191-~191: Ensure spelling is correct
Context: ...추가 옵션과 함께 액티비티 여러 개 제거 ``` pop() 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🔇 Additional comments (4)
docs/pages/docs/get-started/navigating-activities.ko.mdx (4)
138-138: Confirm code fence modifiers order and “/pop/” highlighting supportYou introduced
showLineNumbersand/pop/in the code fence meta. Please verify that:
- The docs theme supports
showLineNumbersin code fence meta.- Inline pattern highlighting with
/pop/is recognized in this position and doesn’t require a different order.If unsupported, the block may render raw meta text or fail to highlight.
150-159: Example addition of goBack/goBackMultiple looks goodThe example clearly demonstrates single and multi-step back navigation with
pop()andpop(3). Nice and readable.To prevent reader confusion, consider adding a short note about behavior when requesting to pop more activities than available (e.g., at root). Is
pop(>length)a no-op or clamped to root? If you want, I can suggest wording after you confirm the intended behavior.
178-181: Examples are accurate and aligned with the description
pop();andpop(3);comments match the behavior explained above.
184-189: Clear illustration of both signatures with optionsGood inclusion of both
pop({ ... })andpop(3, { ... })variants; this resolves the common ambiguity around the first parameter.
|
기여 감사합니다. Workflow가 모두 완료되면 머지하겠습니다. |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
docs/pages/docs/get-started/navigating-activities.ko.mdx (4)
165-166: Minor localization polish for button labels.Korean page currently mixes EN/KR in button labels. Consider localizing for consistency.
Apply this diff if you prefer fully-KR UI strings:
- <button onClick={goBack}>Back</button> - <button onClick={goBackMultiple}>Back 3 Steps</button> + <button onClick={goBack}>뒤로 가기</button> + <button onClick={goBackMultiple}>뒤로 3단계</button>
174-176: Fix spacing and term consistency: ‘첫 번째/두 번째’, and use ‘파라미터’.
- Standard Korean spacing: “첫 번째”, “두 번째”.
- Keep terminology consistent with earlier sections that use “파라미터” instead of “매개변수”.
- Optional: explicitly state that the first parameter is optional.
-`pop()` 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. -첫번째 파라미터를 액티비티 개수로 사용하면, 두번째 매개변수는 추가 옵션을 전달하는 데 사용할 수 있어요. +`pop()` 함수의 첫 번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 있어요. 첫 번째 파라미터는 선택 사항이에요. +첫 번째 파라미터를 액티비티 개수로 사용하면, 두 번째 파라미터는 추가 옵션을 전달하는 데 사용할 수 있어요.
178-189: Make inline comments precise and consistent with counts.Align comments to exact counts and keep numerical style consistent.
-pop(); // 액티비티 하나 제거 +pop(); // 액티비티 1개 제거 -pop(3); // 액티비티 여러 개 제거 +pop(3); // 액티비티 3개 제거 -}); // 추가 옵션과 함께 액티비티 하나 제거 +}); // 추가 옵션과 함께 액티비티 1개 제거 -}); // 추가 옵션과 함께 액티비티 여러 개 제거 +}); // 추가 옵션과 함께 액티비티 3개 제거
201-201: Tighten closing paragraph phrasing.Reduce repetition of “대해서” and use a slightly smoother construction.
-액티비티를 쌓고, 교체하고, 삭제하는 방법에 대해서 알아봤어요. 이제 액티비티 내의 가상의 스택을 만들 수 있는 방법에 대해 알아봐요. +액티비티를 쌓고, 교체하고, 삭제하는 방법을 알아봤어요. 이제 액티비티 내에 가상의 스택을 만드는 방법을 살펴봐요.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
docs/pages/docs/get-started/navigating-activities.ko.mdx(4 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/pages/docs/get-started/navigating-activities.ko.mdx
[grammar] ~174-~174: Ensure spelling is correct
Context: ...xport default Article; ``` pop() 함수의 첫번째 파라미터는 제거할 액티비티의 개수를 지정하거나 추가 옵션을 정의할 수 ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Check the TypeScript typings
🔇 Additional comments (3)
docs/pages/docs/get-started/navigating-activities.ko.mdx (3)
138-138: Confirm code fence directive support for/pop/.The fence adds
showLineNumbers(👍) andcopy /pop/. Please verify that the docs tooling supports/pop/as intended (e.g., search/highlight/focus). If not, use the supported directive (e.g.,highlight=/pop/,focus=/pop/, or remove it).Would you like me to check the English doc counterpart and your site’s MDX fence processing to confirm the correct directive?
150-154: Good illustrative example for single-step back.The helper
goBack()cleanly demonstratespop()with default behavior. Reads well.
155-159: Clear multi-step example.
goBackMultiple()withpop(3)is an effective addition that clarifies the “count” semantics.
2 participants
No description provided.