Skip to content
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

Feature/delivery #82

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Feature/delivery #82

wants to merge 4 commits into from

Conversation

ckrbqja
Copy link
Member

@ckrbqja ckrbqja commented Feb 9, 2025

설명 (Description)

  • ui glustack 으로 점진적 변경

관련 이슈 (Related Issues)

#24

변경 내용 (Changes Made)

체크리스트 (Checklist)

  • iOS에서 테스트 완료
  • Android에서 테스트 완료
  • 웹에서 테스트 완료

리뷰 가이드라인 (Review Guidelines)

  • P1 (Priority 1): 반드시 바로 수정해야 할 항목입니다. 기능에 큰 영향을 주거나, 배포 전에 꼭 해결해야 하는 중요한 사항입니다.

    • 예시: 기능, 성능 오류, 보안 취약점
  • P2 (Priority 2): 수정하면 코드가 더 개선될 수 있는 사항입니다. 큰 이슈는 아니지만, 수정하면 코드가 더 깔끔하고 유지보수가 쉬워집니다.

    • 예시: 코드 스타일, 중복, 비효율
  • P3 (Priority 3): 당장 수정하지 않아도 괜찮지만, 나중에 개선하면 도움이 되는 사항입니다. 기본적으로 모든 리뷰는 P3로 간주해주시면 됩니다.

    • 예시: 리팩토링, 주석 추가/제거, 최적화

@ckrbqja ckrbqja requested a review from a team as a code owner February 9, 2025 02:23
@ckrbqja ckrbqja enabled auto-merge February 9, 2025 02:23
@github-actions github-actions bot added the size/m label Feb 9, 2025
@ckrbqja ckrbqja added feat New feature or request and removed size/m labels Feb 9, 2025
@ckrbqja ckrbqja self-assigned this Feb 9, 2025
@ckrbqja ckrbqja linked an issue Feb 9, 2025 that may be closed by this pull request
import { Button, ButtonSpinner, ButtonText } from "@/components/ui/button";
import { Heading } from "@/components/ui/heading";
import { VStack } from "@/components/ui/vstack";
import useGlobalToast from "@/features/shared/hooks/useGlobalToast";
Copy link
Member

Choose a reason for hiding this comment

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

useGlobalToast 훅이 궁금해서 찾아보니까
파일이 없는 것 같아요

커밋 누락일까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

아 ㅋㅋㅋ 맞아요 커밋 누락이네여
추가했습니다 감사합니다 ㅎㅎ

Copy link
Member

Choose a reason for hiding this comment

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

매의눈... 멋집니다

@github-actions github-actions bot added the size/l label Feb 9, 2025
@github-actions github-actions bot added size/xl and removed size/l labels Feb 9, 2025
Copy link
Contributor

github-actions bot commented Feb 9, 2025

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request size/xl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] delivery layout #22
3 participants