Conversation
#263) [FEAT] 데모데이용 기능 제한
[REFACTOR] ydestreet 폰트 변경
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @jeonbinggu, 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은 데모데이를 위한 기능 제한 배포를 목표로 합니다. 주요 변경 사항은 Ydestreet 폰트 파일 업데이트와 함께, 사진 관리 페이지에서 특정 인화 및 수령 관련 기능을 임시적으로 비활성화하여 데모 환경에 최적화하는 데 중점을 둡니다. 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. Changelog
Activity
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}
| import { getBannerContent } from "@/lib/getBannerContent"; | ||
| import { usePrintOrderStore } from "@/store/usePrintOrder.store"; | ||
| import { usePrintSkip, useConfirmReceipt } from "@/hooks/photoManage"; | ||
| // import { usePrintSkip, useConfirmReceipt } from "@/hooks/photoManage"; |
There was a problem hiding this comment.
사용하지 않는 임포트를 주석 처리로 남겨두기보다 삭제하는 것이 깔끔합니다. 필요한 경우 나중에 Git 히스토리에서 확인하거나 복구할 수 있습니다.
| // const { mutate: printSkip } = usePrintSkip(); | ||
|
|
||
| // 수령 확정 | ||
| const { mutate: confirmReceiptMutate } = useConfirmReceipt(); | ||
| // 수령 확정 - 데모데이용에서는 제거 | ||
| // const { mutate: confirmReceiptMutate } = useConfirmReceipt(); |
There was a problem hiding this comment.
기능 제한을 위해 훅 호출을 주석 처리하셨습니다. 주석 처리된 코드는 유지보수 시 혼란을 줄 수 있으므로 삭제를 권장하며, 기능 제어가 빈번하다면 피처 플래그나 환경 변수를 활용하는 것이 더 체계적인 방법입니다.
| onGoTrackDelivery: () => {}, | ||
| onConfirmReceived: () => | ||
| confirmReceiptMutate(workData.developmentOrderId), | ||
| onConfirmReceived: () => {}, |
There was a problem hiding this comment.
콜백을 빈 함수로 설정하면 사용자가 버튼을 클릭했을 때 아무런 피드백을 받지 못해 시스템 오류로 오해할 수 있습니다. 기능을 제한하려는 의도라면 해당 버튼이 노출되지 않도록 처리하거나, 클릭 시 안내 메시지를 제공하는 것이 좋습니다.
| else { | ||
| setIsDialogOpen(false); | ||
| printSkip(workData.developmentOrderId); | ||
| //printSkip(workData.developmentOrderId); |
There was a problem hiding this comment.
주석 처리된 실행 코드는 삭제하는 것이 좋습니다. 코드의 의도를 명확히 하기 위해 주석 대신 로직 자체를 제거하거나 적절한 처리를 추가해 주세요.
2 participants
🔀 Pull Request Title
데모데이를 위한 기능 제한 배포
🎞️ 주요 코드 설명
주제1
주제2
📌 PR 설명
이번 PR에서 어떤 작업을 했는지 요약해주세요.
📷 스크린샷
UI 변경이 있을 경우 스크린샷을 첨부해주세요.