Skip to content

Conversation

uson1004
Copy link
Member

@uson1004 uson1004 commented Jul 6, 2025

๊ฐœ์š”

์ด๋ฉ”์ผ์„ ์ „์†กํ•˜๋Š” api์˜ ์˜ˆ์™ธ์ฒ˜๋ฆฌ๋ฅผ ํ•˜์ง€ ์•Š์•„ ์•ฑ ํฌ๋ž˜์‰ฌ๊ฐ€ ๋ฐœ์ƒํ•จ

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

  • ์ด๋ฉ”์ผ์„ ์ „์†กํ•˜๋Š” api์˜ ์˜ˆ์™ธ์ฒ˜๋ฆฌ(TooManyRequest)๋ฅผ ์ ์šฉ์‹œ์ผฐ์–ด์š”

์ถ”๊ฐ€ ๋กœ ํ•  ๋ง

  • ๊ณ„์ •์ธ์ฆ Screen์˜ back stack ๊ตฌํ˜„ ํ•„์š”
  • sideEffect๊ฐ€ ํ•œ ๋ฒˆ๋งŒ ๋ฐœ์ƒ์‹œํ‚ค๋„๋ก ๊ตฌํ˜„

@uson1004 uson1004 requested a review from a team July 6, 2025 14:04
@uson1004 uson1004 self-assigned this Jul 6, 2025
@uson1004 uson1004 added the ๐Ÿ‘พbug ๋ฒ„๊ทธ๊ฐ€ ๋ฐœ์ƒํ•œ ๊ฒฝ์šฐ label Jul 6, 2025
}.onSuccess {
postSideEffect(ResetPasswordSideEffect.SendEmailVerificationCodeSuccess)
}.onFailure {
postSideEffect(ResetPasswordSideEffect.EmailVerificationTooManyRequest)
Copy link
Member

Choose a reason for hiding this comment

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

ํŠน์ • SideEffect์—์„œ ๋ชจ๋“  ์—๋Ÿฌ ์ผ€์ด์Šค๋ฅผ ์ฒ˜๋ฆฌํ•˜๊ณ  ํ•˜๊ณ  ์žˆ๊ตฐ์š” ๐Ÿซฃ

Copy link
Member Author

Choose a reason for hiding this comment

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

@aahspringaa4 ๋จผ์ € ๋‹ต๋ณ€์ด ๋งŽ์ด ๋Šฆ์€ ๋ถ€๋ถ„ ์ฃ„์†กํ•ฉ๋‹ˆ๋‹คใ…œ ์ฝ”๋ฉ˜ํŠธ ๋‚จ๊ฒจ์ฃผ์‹  ๋ถ€๋ถ„์ด ์—๋Ÿฌ ์ผ€์ด์Šค ๋ณ„๋กœ SideEffect๋ฅผ ๋‚˜๋ˆ ์•ผํ•œ๋‹ค๋Š” ๋ง์”€์œผ๋กœ ๋‚จ๊ฒจ์ฃผ์‹  ๊ฑธ๋กœ ์ดํ•ดํ–ˆ์Šต๋‹ˆ๋‹ค ๋ง์”€ํ•ด์ฃผ์‹  ๋ถ€๋ถ„์„ ์ œ๊ฐ€ ์ œ๋Œ€๋กœ ์ดํ•ดํ•œ ๊ฒŒ ๋งž์„๊ฐ€์š”? ํ•ด๋‹น ๋ฌธ์ œ์— ๋งž๊ฒŒ ๋กœ์ง์„ ๋ณ€๊ฒฝํ•ด๋ณด๊ณ ์‹ถ์Šต๋‹ˆ๋‹ค๐Ÿ˜Š ์ฒจ์–ธ ์ฃผ์‹œ๋ฉด ๊ฐ์‚ฌํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
๐Ÿ‘พbug ๋ฒ„๊ทธ๊ฐ€ ๋ฐœ์ƒํ•œ ๊ฒฝ์šฐ
Projects
None yet
Development

Successfully merging this pull request may close these issues.

์ด๋ฉ”์ผ ์ธ์ฆ๋ฒˆํ˜ธ ๋ณด๋‚ด๊ธฐ ๋ฒ„๊ทธ
2 participants