docs: v2 sdk forceRedirect, redirectUrl 파라미터 설명 추가 #997
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @MALLLAG, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 v2 SDK의 Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
v2 SDK의 forceRedirect 파라미터에 대한 설명이 추가된 것을 확인했습니다. 사용자 문서(checkout.mdx)와 SDK 스키마(browser-sdk.yml) 파일에 관련 내용이 반영되었습니다. 전반적으로 문서가 명확하게 잘 작성되었으나, forceRedirect 사용 시 redirectUrl 파라미터가 필수인지에 대한 설명이 스키마와 사용자 문서 간에 불일치하는 부분이 있어 수정을 제안했습니다. 해당 부분을 수정하면 사용자의 혼동을 줄이고 더 나은 개발 경험을 제공할 수 있을 것입니다.
| ### 3-2. forceRedirect 옵션으로 리다이렉트 강제하기 | ||
|
|
||
| PC 환경에서도 결과를 리다이렉트 방식으로 받고 싶은 경우 `forceRedirect` 옵션을 사용할 수 있습니다. | ||
| `forceRedirect` 옵션을 사용할 경우 `redirectUrl`을 필수적으로 전달해 주셔야 합니다. |
There was a problem hiding this comment.
forceRedirect 옵션을 사용할 때 redirectUrl이 필수라고 설명되어 있습니다. 하지만 browser-sdk.yml 스키마를 보면 redirectUrl이 없으면 forceRedirect 옵션이 무시되고 Promise로 결과가 반환된다고 정의되어 있습니다. 사용자 혼동을 줄이기 위해 문서의 내용을 SDK의 실제 동작과 일치시키는 것이 좋습니다.
`forceRedirect` 옵션을 사용할 경우 `redirectUrl`을 함께 전달해야 리디렉션이 동작합니다.
|
No description provided.