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

๐Ÿ› fix: igw-55/127 - ์œ ํ•™์ƒ/๊ณ ์šฉ์ฃผ ์„œ๋ฅ˜์ž‘์„ฑ ๋ฒ„๊ทธ ํ•ด๊ฒฐ #128

Merged
merged 12 commits into from
Nov 25, 2024

Conversation

MrMirror21
Copy link
Contributor

Related issue ๐Ÿ› 

#127

Work Description โœ๏ธ

  • ์ž‘์—… ๋‚ด์šฉ
  • ๊ณ ์šฉ์ฃผ/์œ ํ•™์ƒ ์„œ๋ฅ˜ ์ž‘์„ฑ ๊ณผ์ • ์ค‘ ์ž”์กด ์—๋Ÿฌ ๋ฐ UI ์˜ค๋ฅ˜ ์ˆ˜์ •
    • ์„œ๋ฅ˜ ์นด๋“œ ์ ์ƒ‰ ์  ์œ„์น˜ ui ๊นจ์ง ํ˜„์ƒ ์ˆ˜์ •
    • ๊ณ ์šฉ์ฃผ ๊ทผ๋กœ๊ณ„์•ฝ์„œ ๋ฏธ๋ฆฌ๋ณด๊ธฐ ์—ฐ๊ฒฐ(ํ˜„์žฌ ๊ณ ์šฉ์ฃผ๊ฐ€ ์ž‘์„ฑํ•œ ํ›„ ๋ฏธ๋ฆฌ๋ณด๊ธฐ ์‹œ ์„œ๋ฒ„ ์—๋Ÿฌ ๋ฐœ์ƒ)
    • ๊ณ ์šฉ์ฃผ/์œ ํ•™์ƒ ๋’ค๋กœ๊ฐ€๊ธฐ url ๋ณ€๋™ ์ ์šฉ
    • ์›”๊ธ‰์ผ ์ž…๋ ฅ Input prefix ui ์ค„ ๋ฐ”๊ฟˆ ํ˜„์ƒ ์ˆ˜์ •
    • ๊ณ ์šฉ์ฃผ ๊ทผ๋กœ๊ณ„์•ฝ์„œ ์ž‘์„ฑ ํ›„ ๋ฆฌ๋‹ค์ด๋ ‰ํŠธ ๊ฒฝ๋กœ ์ •์ •
    • ์œ ํ•™์ƒ์ด ์ž…๋ ฅํ•œ ์ •๋ณด ์ค‘ ์„œ๋ช… ๋ฏธ๋ฆฌ๋ณด๊ธฐ๊ฐ€ ์ •์ƒ์ ์œผ๋กœ ํ‘œ๊ธฐ๋˜๋„๋ก ์ˆ˜์ •
    • ๋ณดํ—˜ ์„ ํƒ ์‹œ enum์œผ๋กœ ์ž…๋ ฅ๋˜๋„๋ก ์ˆ˜์ •
    • ๊ณ ์šฉ์ฃผ ๊ทผ๋กœ๊ณ„์•ฝ์„œ ๋ฏธ๋ฆฌ๋ณด๊ธฐ ๊ด€๋ จ ์—…๋ฐ์ดํŠธ ๋œ api ๋ช…์„ธ์„œ type ์ ์šฉ

Uncompleted Tasks ๐Ÿ˜…

  • ๊ทผ๋กœ๊ณ„์•ฝ์„œ ๋ฏธ๋ฆฌ๋ณด๊ธฐ ๊ด€๋ จ ์„œ๋ฒ„ ์—๋Ÿฌ ๋ฐœ์ƒ์œผ๋กœ ์ดํ›„ submit๊ณผ confirm ํ…Œ์ŠคํŠธ ๋Œ€๊ธฐ ์ƒํƒœ

To Reviewers ๐Ÿ“ข

@MrMirror21 MrMirror21 added ๐Ÿž BugFix Something isn't working ๐ŸŽจ Html&CSS Markup & Styles ๐Ÿ“ญ API Server API communication labels Nov 24, 2024
@MrMirror21 MrMirror21 self-assigned this Nov 24, 2024
@MrMirror21 MrMirror21 changed the title Fix/igw 55/127 ๐Ÿ› fix: igw-55/127 - ์œ ํ•™์ƒ/๊ณ ์šฉ์ฃผ ์„œ๋ฅ˜์ž‘์„ฑ ๋ฒ„๊ทธ ํ•ด๊ฒฐ Nov 24, 2024
Copy link
Contributor

@hyeona01 hyeona01 left a comment

Choose a reason for hiding this comment

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

์ž…๋ ฅ๊ฐ’์ด ์ •~ ๋ง ๋‹ค์–‘ํ•˜๊ณ  ๋งŽ๋„ค์š”
๊ณ ์ƒ ๋งŽ์œผ์…จ์Šต๋‹ˆ๋‹ค!!!

@MrMirror21 MrMirror21 merged commit d8a1d8d into main Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
๐Ÿ“ญ API Server API communication ๐Ÿž BugFix Something isn't working ๐ŸŽจ Html&CSS Markup & Styles
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants