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

[CLIENT] Week4 #239

Merged
merged 191 commits into from
Dec 2, 2022
Merged

[CLIENT] Week4 #239

merged 191 commits into from
Dec 2, 2022

Conversation

mjsdo
Copy link
Member

@mjsdo mjsdo commented Dec 2, 2022

Issues

Week4 병합

mjsdo and others added 30 commits December 2, 2022 21:38
response.data -> response.data.result
주석 달았으니 사용할 때 참고
채널 페이지 추가하고, 커뮤니티의 하위 계층으로 변경

변경이유: 커뮤니티 입장시 곧바로 채널로 들어가는게 아니므로, 디폴트로 보여줘야 할 페이지가 필요함.
아이디가 아닌 커뮤니티 이름을 보여주도록 변경
추가함에 따라 반환 타입이 달라져서 이 훅을 사용하는 컴포넌트 리팩토링
leegwae and others added 27 commits December 2, 2022 21:38
…channelId`의 `users` 필드 사용하도록 변경 #213
api 명세 변경에 따라 업데이트
@mjsdo mjsdo merged commit cd0cb0e into dev Dec 2, 2022
@mjsdo mjsdo added this to the ⛰️ Week4 milestone Dec 5, 2022
@mjsdo mjsdo self-assigned this Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants