-
Couldn't load subscription status.
- Fork 3
QI1-207-Footer-수정 #107
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
QI1-207-Footer-수정 #107
The head ref may contain hidden characters: "QI1-207-Footer-\uC218\uC815"
Conversation
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.
Summary of Changes
Hello @yshls, 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!
I've updated the application's footer components for both desktop and mobile views. The primary goal was to enhance navigation by updating existing links and introducing new ones, while also implementing dynamic visibility for certain links based on user authentication status and administrative roles. Additionally, I've refreshed the 'Company Information' section to display 'Team Member' details instead, along with some minor branding and cleanup.
Highlights
- Enhanced Navigation: Updated existing footer links with correct destinations and added new links for "Blog" and "Admin" pages.
- Dynamic Link Visibility: Implemented logic to show "My Page" only to logged-in users and "Admin" link exclusively to users with an 'ADMIN' role, leveraging useAuthStore.
- Team Information Update: Replaced the generic "Company Information" section in the footer with a detailed list of "Team Members" (Frontend and Backend).
- Branding and Cleanup: Updated the "Snac" logo text to "SNAC" and removed outdated privacy/terms links from the footer.
Using Gemini Code Assist
The 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 in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.
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 or fill out our survey to provide feedback.
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
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
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.
Code Review
푸터 컴포넌트 수정에 대한 코드 리뷰입니다. 전반적으로 데스크톱과 모바일 푸터의 UI를 개선하고, 사용자 인증 상태에 따라 동적으로 링크를 표시하도록 변경한 점이 좋습니다. 리뷰 결과, 관리자 링크가 표시되지 않는 심각한 버그와 함께 스타일 가이드에 맞지 않는 상수 네이밍이 발견되었습니다. 자세한 내용은 아래 리뷰 코멘트를 확인해주세요. 이번 리뷰에서는 Toss TypeScript 스타일 가이드의 네이밍 컨벤션 규칙을 참조하여 피드백을 드렸습니다.
|
ㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡㅡ |
|
/gemini help |
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.
Code Review
이 PR은 사용자의 로그인 상태와 역할에 따라 푸터에 표시되는 링크를 동적으로 변경하는 내용을 담고 있습니다. DesktopFooter와 MobileFooter 컴포넌트에 동일한 로직이 적용되었습니다. 전반적으로 기능은 잘 구현되었지만, 코드 중복, 가독성, 그리고 스타일 가이드 준수 측면에서 몇 가지 개선할 점이 보입니다. 주요 피드백으로는 중복된 링크 필터링 로직을 유틸리티 함수로 분리하고, 필터링 로직을 switch 문으로 리팩토링하며, 스타일 가이드에 따라 상수 네이밍을 수정하는 것을 제안합니다. 이러한 변경을 통해 코드의 유지보수성과 일관성을 높일 수 있을 것입니다.
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 in your pull request via creating an issue comment (i.e. comment on the pull request page) 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 or fill out our survey to provide feedback. |
|
It looks like you were trying to initiate a code review. If you'd like me to perform a code review for this pull request, please use the |
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 in your pull request via creating an issue comment (i.e. comment on the pull request page) 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 or fill out our survey to provide feedback. |
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.
LGTM
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.
굿
📝 변경 사항
홈, 실시간 매칭, 블로그 만 보입니다.
홈, 실시간 매칭, 블로그, 문의하기 만 보입니다.
2.1. 로그인 한 경우: 관리자일때,
홈, 실시간 매칭, 블로그, 문의하기, 관리자 만 보입니다.
🔍 변경 사항 세부 설명
🕵️♀️ 요청사항
📷 스크린샷 (선택)