Skip to content

Conversation

@ohprettyhak
Copy link
Contributor

@ohprettyhak ohprettyhak commented Mar 19, 2024

🌍 이슈 번호

✅ 작업 내용

  • 매니저 앱의 대회 내부의 경기 관련 페이지를 구현했습니다.
    • 경기 리스트
    • 경기 삭제
    • 경기 생성

📝 참고 자료

  • 현재 팀 페이지가 없어 경기 생성 중 막힙니다. 팀 페이지 만들어서 올리면 정상적으로 확인 가능할 거에요!

♾️ 기타

  • 모달의 경우 현재 중복으로 사용되는 부분이 많습니다. 기존에 얘기 나온것 처럼 한 번에 추상화 하면 좋을 거 같습니다.
  • 카드 레이아웃이 잘 적립되어 있지만, 이 중에서도 아래와 같이 중복으로 사용되는 카드가 있습니다.
image image
  • 이 부분들도 추후에 추상화 대상에 포함하면 좋을 듯 합니다.

@ohprettyhak ohprettyhak added the feature New features label Mar 19, 2024
@ohprettyhak ohprettyhak self-assigned this Mar 19, 2024
@ohprettyhak ohprettyhak linked an issue Mar 19, 2024 that may be closed by this pull request
2 tasks
@netlify
Copy link

netlify bot commented Mar 19, 2024

Deploy Preview for hufscheer ready!

Name Link
🔨 Latest commit 23bfdd1
🔍 Latest deploy log https://app.netlify.com/sites/hufscheer/deploys/65f9ab78e364460008e56f30
😎 Deploy Preview https://deploy-preview-109--hufscheer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 53
Accessibility: 71
Best Practices: 83
SEO: 55
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Mar 19, 2024

Deploy Preview for hufscheer-manager ready!

Name Link
🔨 Latest commit 23bfdd1
🔍 Latest deploy log https://app.netlify.com/sites/hufscheer-manager/deploys/65f9ab78e55ac100087ece8a
😎 Deploy Preview https://deploy-preview-109--hufscheer-manager.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@seongminn seongminn left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

Comment on lines +17 to +20
':hover': {
backgroundColor: theme.colors.gray[2],
color: theme.colors.primary[3],
},
Copy link
Member

Choose a reason for hiding this comment

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

selectors 프로퍼티를 이용해야 하는 줄 알았는데 아닌가보네요!👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://vanilla-extract.style/documentation/styling/ 해당 페이지에서 hover를 위와 같이 사용하더라구요!

Copy link
Contributor Author

@ohprettyhak ohprettyhak Mar 19, 2024

Choose a reason for hiding this comment

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

말씀하신 부분은 모두 적용해놓았습니다. 코드를 쭉 보니까 쿼리 혹은 뮤테이션에 파라미터를 넣을 때 string, number 타입이 혼재되어 사용되더라구요. 이 부분도 추후에 통일하면 더욱 좋을 거 같습니다. 따로 노션에 정리하겠습니다.

ohprettyhak and others added 3 commits March 20, 2024 00:02
Co-authored-by: Seongmin Choi <88662637+seongminn@users.noreply.github.com>
Co-authored-by: Seongmin Choi <88662637+seongminn@users.noreply.github.com>
@ohprettyhak ohprettyhak merged commit e2f8e13 into main Mar 19, 2024
@ohprettyhak ohprettyhak deleted the feat/manager/league-game branch March 19, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 매니저 앱 대회 경기 관리 구현

3 participants