Skip to content
This repository has been archived by the owner on Dec 7, 2024. It is now read-only.

๐Ÿ”€:: clubMember-api-change #148

Merged
merged 22 commits into from
Feb 9, 2023
Merged

Conversation

minaje0917
Copy link
Member

@minaje0917 minaje0917 commented Feb 7, 2023

๊ฐœ์š”

clubmember API๋ฅผ API ๋ช…์„ธ์„œ์— ๋งž๊ฒŒ ๋ณ€๊ฒฝํ•˜์˜€์Šต๋‹ˆ๋‹ค.

์ž‘์—…์‚ฌํ•ญ

  • clubAPI, clubMemberAPI ๋ถ„๋ฆฌ
  • ClubMemberError ์ถ”๊ฐ€
  • ClubRemote, ClubMemberRemote ๋ถ„๋ฆฌ
  • ClubRepository, ClubMemberRepository ๋ถ„๋ฆฌ
  • add DefaultClubMemberRepository

๋ณ€๊ฒฝ๋กœ์ง

๋ณ€๊ฒฝ์ „

๋ณ€๊ฒฝํ›„

์‚ฌ์šฉ๋ฐฉ๋ฒ•

๊ธฐํƒ€

@minaje0917 minaje0917 linked an issue Feb 7, 2023 that may be closed by this pull request
@minaje0917 minaje0917 changed the title ๐Ÿ”€:: clubmember api change ๐Ÿ”€:: clubMember-api-change Feb 7, 2023
@minaje0917 minaje0917 added 1๏ธโƒฃPriority: High ์šฐ์„ ์ˆœ์œ„ ์ƒ ๐Ÿ”จ Refactor ์ฝ”๋“œ ๋ฆฌํŒฉํ† ๋ง labels Feb 7, 2023
@minaje0917 minaje0917 requested a review from baekteun February 7, 2023 11:09
Copy link
Member

@baekteun baekteun left a comment

Choose a reason for hiding this comment

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

UseCase๋„ ๋ณ€๊ฒฝํ•„์š”ํ•ฉ๋‹ˆ๋‹ค

@minaje0917 minaje0917 merged commit 60c116f into master Feb 9, 2023
@minaje0917 minaje0917 deleted the 145-clubmember-api-change branch February 9, 2023 07:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1๏ธโƒฃPriority: High ์šฐ์„ ์ˆœ์œ„ ์ƒ ๐Ÿ”จ Refactor ์ฝ”๋“œ ๋ฆฌํŒฉํ† ๋ง
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ClubMember API ์ •์ƒํ™”
3 participants