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

reaction-speed-test 기록 페이지 supabase 연동 #76

Merged
merged 8 commits into from
Feb 9, 2025

Conversation

JAM-PARK
Copy link
Member

설명 (Description)

  • supabase records 테이블 생성 👉 시작하기~insert, 기록보기~select 쿼리 적용

관련 이슈 (Related Issues)

스크린샷/동영상 (Screenshots/Videos)

image image

테스트 방법 (How to Test)

  • /apps/reaction-speed-test에서 pnpm run start

체크리스트 (Checklist)

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

리뷰 가이드라인 (Review Guidelines)

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

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

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

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

@JAM-PARK JAM-PARK added feat New feature or request size/s labels Jan 25, 2025
@JAM-PARK JAM-PARK self-assigned this Jan 25, 2025
@JAM-PARK JAM-PARK requested a review from a team as a code owner January 25, 2025 05:03
@github-actions github-actions bot added size/m and removed size/s labels Jan 25, 2025
Copy link
Contributor

@Lim-Ji-Hyeon Lim-Ji-Hyeon left a comment

Choose a reason for hiding this comment

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

오!! supabase를 이용해 기록한 데이터를 저장하고 UI에 표시하는 것까지 구현하셨네요!! 👍

@JAM-PARK JAM-PARK force-pushed the feature/reaction-speed-test-record branch from a758829 to c76f7f4 Compare January 27, 2025 03:47
@JAM-PARK JAM-PARK force-pushed the feature/reaction-speed-test-record branch from f1f95d7 to 87f36d7 Compare February 9, 2025 06:58
- main rebase 이후 로컬 앱 시작 시 env 에러 발생
Error: Error reading Expo config at /app-factory/apps/reaction-speed-test/app.config.ts:
Required environment variable EXPO_PUBLIC_SUPABASE_URL is missing
- 관련 커밋 a58d9e5
web-publish에서 env를 적용하기 위함
@JAM-PARK JAM-PARK merged commit 614ef40 into main Feb 9, 2025
3 checks passed
@JAM-PARK JAM-PARK deleted the feature/reaction-speed-test-record branch February 9, 2025 08:16
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/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants