Conversation
Summary of ChangesHello @dle234, 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은 댓글의 전체 공개 여부에 따라 블러 처리 기능을 구현합니다. 새로운 API를 통해 댓글 전체 공개 상태를 확인하고, 이 상태에 따라 개별 댓글의 블러 처리 여부를 동적으로 제어하여 사용자 경험을 개선합니다. 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}
| const { data: isCommentDisclosed } = useQuery( | ||
| ["commentDisclosure"], | ||
| () => getCommentDisclosure(), | ||
| { | ||
| initialData: false, | ||
| } | ||
| ); |
There was a problem hiding this comment.
isCommentDisclosed를 가져오는 쿼리에 initialData: false가 설정되어 있어, API 응답이 오기 전까지는 항상 블러 처리된 상태로 렌더링됩니다. 이로 인해 실제로는 공개 상태임에도 불구하고 아주 짧은 시간 동안 블러 처리가 보였다가 사라지는 깜빡임 현상이 발생할 수 있습니다. 또한, 이 쿼리의 로딩 및 에러 상태가 하단의 if (!data || isLoading) 및 if (error) 블록에 포함되지 않아, 해당 API 호출이 지연되거나 실패했을 때의 처리가 누락되어 있습니다.
| </button> | ||
| </div> | ||
| {comment.isBlurred ? ( | ||
| {!isCommentDisclosed && comment.isBlurred ? ( |
There was a problem hiding this comment.
96번 라인에서 댓글의 블러 처리 조건에 isCommentDisclosed가 추가되었으나, 좋아요 버튼의 활성화 조건(88번 라인)에는 반영되지 않았습니다. 전체 공개 상태(isCommentDisclosed가 true)여서 댓글 내용이 보임에도 불구하고, 원본 데이터상 isBlurred가 true라면 좋아요를 누를 수 없는 불일치가 발생합니다. 버튼의 활성화 로직도 동일한 조건으로 수정이 필요합니다.
There was a problem hiding this comment.
요구 사항을 보니 해당 api는 회장단만이 사용할 수 있습니다.
해당 기능은 모두가 보는 커맨트와 결합시키는게 아닌 회장단이 사용하는 아래 api를 위한 조회 api 인 것 같습니다!
POST /api/v1/comments/disclosure
그래서 회장단이 현재 댓글 조회 상태를 조절할수있는 페이지가 따로 필요할 것 같습니다!
관리자페이지가 괜찮을 것 같은데 확인해보니 회장단 role은 현재 관리자페이지에 접근하지 못하네요
분기처리해서 회장단페이지를 새로 만드는식으로 해야할 것 같다는 생각이드네요..! 해당 부분은 같이 고민해보죠
제가 잘못파악한 부분이 있으면 말해주세요-!
제가 잘못 이해했네요ㅠㅠ 일단 이 PR은 close 하도록 하겠습니다! 회의 후에 추가하는게 좋을 것 같네요 |
2 participants
관련 이슈
작업 분류
PR을 통해 해결하려는 문제가 무엇인가요? 🚀
체크리스트 ✅
reviewers설정assignees설정label설정