-
Notifications
You must be signed in to change notification settings - Fork 2
[FEAT] 리그 정보 관리 페이지 생성 #93
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
Conversation
✅ Deploy Preview for hufscheer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for hufscheer-manager ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
ohprettyhak
left a comment
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.
고생하셨습니다!
apps/manager/api/league.ts
Outdated
|
|
||
| export const putLeague = async (data: PutLeaguePayload) => { | ||
| export const updateLeague = async (data: UpdateLeaguePayload) => { | ||
| await instance.put('/league/', data); |
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.
| await instance.put('/league/', data); | |
| await instance.put('/leagues/', data); |
API 문서와 실제 호출 링크가 달라 404 오류가 발생하네요.
| /> | ||
| </Flex> | ||
|
|
||
| <Flex direction="column" mt="md" gap={rem(4)}> |
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.
혹시 리그 정보 페이지에서는 종목 추가가 불가능할까요? 현재는 만들어진 종목의 수정만 가능한 거 같습니다!
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.
아직 개발 중이라 올리지 않았는데, 추가할 수 있도록 수정하고 있습니다!
| { value: '2', label: '농구' }, | ||
| { value: '3', label: '야구' }, | ||
| ]} | ||
| data={GAMES.SPORTS} |
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.
상수로 적립하여 사용하는 부분은 좋은 거 같습니다! 이쪽은 오히려 API 호출이 없는 것이 더 낫겠네요.
| @@ -0,0 +1,9 @@ | |||
| import useLeagueQuery from './useLeagueQuery'; | |||
|
|
|||
| export default function useLeagueDetailQuery(leagueId: number) { | |||
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.
이 쿼리는 /report 페이지에서도 사용하겠습니다 ㅎㅎ
| leagueId: league.leagueId, | ||
| }), | ||
| ); | ||
| }, [buttonRef, edit, form, league.leagueId, mutateUpdateLeague]); |
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.
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.
이건 로컬에서 동작하는 것과 다르게 동작하고 있네요🤔 한 번 원인을 찾아보겠습니다!
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.
이제 정상적으로 동작합니다!
ohprettyhak
left a comment
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.
고생하셨습니다. 마지막으로 데이터 저장 후 동기화 관련해서 코멘트 남깁니다!
| return ( | ||
| <Layout navigationTitle={'대회 정보 관리'} navigationMenu={<RightButton />}> | ||
| <LeagueDetailForm | ||
| league={league} |
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.
league를 직접 넘겨주다보니 편집 후에도 메인에는 동기화되지 않는 문제가 발생하네요.
리그 관리 페이지 → 대회 정보 관리 → 오른쪽 편집 버튼 클릭 후 날짜 수정 → 뒤로가기를 통해 리그 관리 페이지 → 대회 정보 관리
순으로 진행해보시면 실제 서버에서 가지고 있는 값은 바뀌었지만, 리그 관리 페이지에서 전달하는 league 값이 바뀌지 않아 이전 데이터를 불러오는 것을 확인할 수 있습니다.
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.
확인했고 수정해서 올렸습니다! 감사합니다ㅎㅎ 👍👍
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.
머지해도 좋을 거 같습니다! 고생하셨습니다 👍


🌍 이슈 번호
✅ 작업 내용
📝 참고 자료
♾️ 기타