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

๐Ÿ”€ :: [#195] ๊ฐœ์„ค์‹ ์ฒญ์„œ ๊ด€๋ จ ์ฝ”๋“œ ์‚ญ์ œ #196

Merged
merged 19 commits into from
Dec 28, 2023

Conversation

junseopark0331
Copy link
Contributor

๊ฐœ์š”

๊ฐœ์„ค์‹ ์ฒญ์„œ ๊ด€๋ จ ์ฝ”๋“œ๋ฅผ ์‚ญ์ œํ•ฉ๋‹ˆ๋‹ค.

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

MemberAppend, NewClub, Update ํŒŒ์ผ์˜ VC์™€ Reactor๋ถ€๋ถ„์„ ์‚ญ์ œํ–ˆ์Šต๋‹ˆ๋‹ค.
๋™์•„๋ฆฌ ๊ฐœ์„ค ๊ด€๋ จ API๋ฅผ ์‚ญ์ œํ–ˆ์Šต๋‹ˆ๋‹ค.

์งˆ๋ฌธ์‚ฌํ•ญ

  1. Update ๊ทธ๋ฃน ๊ฐ™์€๊ฒฝ์šฐ ๋™์•„๋ฆฌ๋ฅผ ์ˆ˜์ •ํ•˜๋Š” ์ฝ”๋“œ๋กœ์„œ ์ง€์šฐ์ง€ ์•Š๊ณ  ๊ทธ๋ž˜๋„ ๋‘ฌ์•ผํ•  ๊ฒƒ ๊ฐ™์€๋ฐ ์ง€์›Œ๋„ ๊ดœ์ฐฎ์€๊ฐ€์š” ??
  2. api์™€ response ๋ถ€๋ถ„ ์ฝ”๋“œ์ค‘ ์ง€์›Œ์•ผํ•  ์ฝ”๋“œ๊ฐ€ ๋” ์žˆ๋‹ค๋ฉด ์•Œ๋ ค์ฃผ์‹œ๋ฉด ๊ฐ์‚ฌํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค

@junseopark0331 junseopark0331 added 1๏ธโƒฃPriority: High ์šฐ์„ ์ˆœ์œ„ ์ƒ ๐Ÿ”จ Refactor ์ฝ”๋“œ ๋ฆฌํŒฉํ† ๋ง labels Dec 27, 2023
@junseopark0331 junseopark0331 linked an issue Dec 27, 2023 that may be closed by this pull request
@baekteun
Copy link
Member

ํ™ˆํ™”๋ฉด์—์„œ ๊ฐœ์„คํ•˜๋Ÿฌ๊ฐ€๋Š” ๋ฒ„ํŠผ๋„ ์ง€์›Œ์•ผํ• ๊ฑฐ๊ฐ™์•„์š”

@baekteun baekteun changed the title ๐Ÿ”€ :: ๊ฐœ์„ค์‹ ์ฒญ์„œ ๊ด€๋ จ ์ฝ”๋“œ ์‚ญ์ œ ๐Ÿ”€ :: [#195] ๊ฐœ์„ค์‹ ์ฒญ์„œ ๊ด€๋ จ ์ฝ”๋“œ ์‚ญ์ œ Dec 27, 2023
@junseopark0331
Copy link
Contributor Author

ํ™ˆํ™”๋ฉด์—์„œ ๊ฐœ์„คํ•˜๋Ÿฌ๊ฐ€๋Š” ๋ฒ„ํŠผ๋„ ์ง€์›Œ์•ผํ• ๊ฑฐ๊ฐ™์•„์š”

90f1d1a ์‚ญ์ œํ–ˆ์Šต๋‹ˆ๋‹ค !

@uuuunseo
Copy link
Member

๊ณต๋ฐฑ์ด ์•„์ง ๋œ ์ง€์›Œ์ง„ ํŒŒ์ผ์ด ์žˆ๋Š” ๊ฑฐ ๊ฐ™์•„์š”!!

Comment on lines 53 to 55
func configureView() {

}
Copy link
Member

Choose a reason for hiding this comment

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

์ด ํ•จ์ˆ˜๋Š” ๋น„์–ด์žˆ๋Š”๋ฐ ์–ด๋–จ ๋•Œ ์‚ฌ์šฉํ•˜๋Š” ๊ฑด๊ฐ€์š”?? ์‚ฌ์šฉ ์•ˆ ํ•˜๋ฉด ์ง€์›Œ๋„ ๋˜์ง€ ์•Š์„๊นŒ์š”?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

17c9cfe

์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค !

@junseopark0331
Copy link
Contributor Author

๊ณต๋ฐฑ์ด ์•„์ง ๋œ ์ง€์›Œ์ง„ ํŒŒ์ผ์ด ์žˆ๋Š” ๊ฑฐ ๊ฐ™์•„์š”!!

์–ด... ์ผ๋‹จ ๋‚˜๋ฆ„ ์ง€์›Œ๋ณด๊ธด ํ–ˆ๋Š”๋ฐ ์ถ”๊ฐ€๋กœ ๊ณต๋ฐฑ ์ง€์›Œ์•ผํ•˜๋Š” ํŒŒ์ผ์ด ์žˆ๋‹ค๋ฉด ์•Œ๋ ค์ฃผ์‹ค ์ˆ˜ ์žˆ์œผ์‹ค๊นŒ์š”??

@uuuunseo
Copy link
Member

๊ณต๋ฐฑ์ด ์•„์ง ๋œ ์ง€์›Œ์ง„ ํŒŒ์ผ์ด ์žˆ๋Š” ๊ฑฐ ๊ฐ™์•„์š”!!

์–ด... ์ผ๋‹จ ๋‚˜๋ฆ„ ์ง€์›Œ๋ณด๊ธด ํ–ˆ๋Š”๋ฐ ์ถ”๊ฐ€๋กœ ๊ณต๋ฐฑ ์ง€์›Œ์•ผํ•˜๋Š” ํŒŒ์ผ์ด ์žˆ๋‹ค๋ฉด ์•Œ๋ ค์ฃผ์‹ค ์ˆ˜ ์žˆ์œผ์‹ค๊นŒ์š”??

ClubTypeCardButton.swift, NewClubStepView.swift, NewClubSteppedProgressBar.swift, NewClubTextField.swift
์œ„ ํŒŒ์ผ์— ๋ถˆํ•„์š”ํ•œ ๊ณต๋ฐฑ์ด ์žˆ์–ด์š”!!

@junseopark0331
Copy link
Contributor Author

๊ณต๋ฐฑ์ด ์•„์ง ๋œ ์ง€์›Œ์ง„ ํŒŒ์ผ์ด ์žˆ๋Š” ๊ฑฐ ๊ฐ™์•„์š”!!

์–ด... ์ผ๋‹จ ๋‚˜๋ฆ„ ์ง€์›Œ๋ณด๊ธด ํ–ˆ๋Š”๋ฐ ์ถ”๊ฐ€๋กœ ๊ณต๋ฐฑ ์ง€์›Œ์•ผํ•˜๋Š” ํŒŒ์ผ์ด ์žˆ๋‹ค๋ฉด ์•Œ๋ ค์ฃผ์‹ค ์ˆ˜ ์žˆ์œผ์‹ค๊นŒ์š”??

ClubTypeCardButton.swift, NewClubStepView.swift, NewClubSteppedProgressBar.swift, NewClubTextField.swift ์œ„ ํŒŒ์ผ์— ๋ถˆํ•„์š”ํ•œ ๊ณต๋ฐฑ์ด ์žˆ์–ด์š”!!

ebadb83 ์ˆ˜์ •ํ–ˆ์–ด์š” !

@junseopark0331 junseopark0331 merged commit 4400fb3 into master Dec 28, 2023
4 checks passed
@junseopark0331 junseopark0331 deleted the 195-delete-CreateClubCode branch December 28, 2023 02:50
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.

๊ฐœ์„ค์‹ ์ฒญ์„œ ์‚ญ์ œ
3 participants