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

feat: Mock API #23

Merged
merged 19 commits into from
Jun 23, 2024
Merged

feat: Mock API #23

merged 19 commits into from
Jun 23, 2024

Conversation

devmizz
Copy link
Contributor

@devmizz devmizz commented Jun 16, 2024

๐Ÿ˜‹ ์ž‘์—…ํ•œ ๋‚ด์šฉ

  • ํ˜„์žฌ ๋‚˜์˜จ ๊ธฐ๋Šฅ์ •์˜์„œ ๊ธฐ์ค€์œผ๋กœ ํด๋ผ์ด์–ธํŠธ์—๊ฒŒ ์ œ๊ณตํ•  ๋ชฉ API ์ž‘์„ฑ

๐Ÿ™ PR Point

  • ์—”๋“œํฌ์ธํŠธ๊ฐ€ RESTfulํ•œ์ง€
  • Security ์„ค์ •์€ ๊ดœ์ฐฎ์€์ง€

๐Ÿ‘ ๊ด€๋ จ ์ด์Šˆ


@devmizz devmizz added the feat add feature label Jun 16, 2024
@devmizz devmizz requested a review from GaBaljaintheroom June 16, 2024 16:39
@devmizz devmizz self-assigned this Jun 16, 2024
@devmizz devmizz changed the title feat: ๋กœ๊ทธ์ธ, ๋กœ๊ทธ์•„์›ƒ ๋ชฉ ์ปจํŠธ๋กค๋Ÿฌ feat: Mock API Jun 16, 2024
Copy link
Contributor

@GaBaljaintheroom GaBaljaintheroom left a comment

Choose a reason for hiding this comment

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

show-api ๋ชจ๋“ˆ์•ˆ์— artist, genre, show ๋„๋ฉ”์ธ์— ๊ด€ํ•œ API๋“ค์ด ๋ชจ๋‘ ๋“ค์–ด ์žˆ๋Š”๋ฐ, ๊ฐ ๋„๋ฉ”์ธ์ด ์„œ๋กœ ์—ฐ๊ด€์ด ๋งŽ์ด ๋˜์–ด์žˆ๊ธฐ ๋•Œ๋ฌธ์— ๊ฐ๊ฐ API ๋ชจ๋“ˆ๋กœ ๋ถ„๋ฆฌ๋ฅผ ์•ˆ ํ•˜์‹ ๊ฑด์ง€ ๊ถ๊ธˆํ•ด์š”!

๋งŽ์€ ์–‘์ด์—ˆ์„ ํ…๋ฐ ์ˆ˜๊ณ  ๋งŽ์œผ์…จ์Šต๋‹ˆ๋‹ค!!!

devmizz added 19 commits June 22, 2024 01:54
@devmizz devmizz force-pushed the feat-mock-controller branch from 0c98bb7 to cab0f1c Compare June 21, 2024 16:55
@devmizz devmizz merged commit ae8d468 into develop Jun 23, 2024
1 check passed
@devmizz devmizz deleted the feat-mock-controller branch June 23, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat add feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants