Skip to content

Conversation

@hyonun321
Copy link
Contributor

📝 변경 사항

  1. 블로그 목차 제거
  2. 실시간 매칭 거래 시간 50분->10분으로 변경

🔍 변경 사항 세부 설명

🕵️‍♀️ 요청사항

📷 스크린샷 (선택)

@hyonun321 hyonun321 self-assigned this Aug 7, 2025
@hyonun321 hyonun321 added the CHORE 그외 기타 label Aug 7, 2025
@github-actions github-actions bot enabled auto-merge August 7, 2025 02:23
Copy link

@gemini-code-assist gemini-code-assist bot left a 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 @hyonun321, 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!

This pull request primarily focuses on enhancing the user experience and clarity of the homepage by refining its content and messaging. Additionally, it includes a functional update to the real-time trading feature, extending the transaction duration.

Highlights

  • Blog Content Management: The Table of Contents (TOC) functionality has been completely removed from blog posts, simplifying the blog post display.
  • Homepage Content Refinement: Extensive updates have been made to the homepage's introductory sections, including revised marketing copy, updated feature descriptions, and adjusted sample trade item data. These changes aim to better articulate SNAC's value propositions, such as personal information protection, real-time matching, and the point system.
  • Real-time Trading Duration: The time limit for real-time matching transactions has been increased from 5 minutes to 10 minutes, providing users with more time to complete their trades.
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

  1. 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.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

블로그 게시물에서 목차(TOC) 기능을 제거하고, 실시간 매칭 거래의 제한 시간을 50분에서 10분으로 단축하는 변경 사항이 있습니다. 또한, 홈페이지의 소개 문구를 개선하여 서비스의 장점을 더 명확하게 전달하도록 수정되었습니다. 거래 제한 시간을 상수로 추출하여 코드의 가독성을 높이는 것을 제안합니다.

const { partner, setUserRole, userRole, updatePartner } = useMatchStore();
const [currentStep, setCurrentStep] = useState<TradingStep>('confirmation');
const [timeLeft, setTimeLeft] = useState(3000); // 5분 제한
const [timeLeft, setTimeLeft] = useState(600); // 10분 제한

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

거래 제한 시간을 10분으로 변경하신 부분은 좋습니다. 다만, 600이라는 값은 '매직 넘버'로 사용되어 코드의 가독성을 저해할 수 있습니다. 의미 있는 이름의 상수로 추출하여 사용하는 것을 권장합니다. 예를 들어, 컴포넌트 외부에 다음과 같이 상수를 정의하고1, useState에서 이 상수를 사용하면 코드의 의도가 명확해집니다.

const TRADING_TIME_LIMIT_SECONDS = 600; // 10분
const [timeLeft, setTimeLeft] = useState(TRADING_TIME_LIMIT_SECONDS);

Style Guide References

Footnotes

  1. 상수는 UPPER_SNAKE_CASE를 사용합니다. (link)

@github-actions github-actions bot merged commit 3d7c3ef into main Aug 7, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CHORE 그외 기타

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants