Skip to content

Comments

Fix/register next#2

Merged
Sangjun-man merged 2 commits intoqafrom
fix/register_next
Sep 16, 2023
Merged

Fix/register next#2
Sangjun-man merged 2 commits intoqafrom
fix/register_next

Conversation

@Sangjun-man
Copy link
Contributor

์Šคํฌ๋ฆฐ์ƒท


Motivation

qa ๊ด€๋ จ ๋‘๊ฐ€์ง€ ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค

  • ์ปดํฌ๋„ŒํŠธ์˜ onSubmit props์— void ํ•จ์ˆ˜(()=>{}) ์ถ”๊ฐ€
  • ๋ด‰์‚ฌ์ž ํšŒ์›๊ฐ€์ž… ํ›„ ๋กœ๊ทธ์ธ ํ•˜๋„๋ก ์ˆ˜์ •

To Reviewers

  - onSubmit์œผ๋กœ Props๊ฐ€ ์ „๋‹ฌ๋˜์ง€ ์•Š๋Š” ๊ฒฝ์šฐ, form ์ž…๋ ฅํ•  ๋•Œ ์—๋Ÿฌ๊ฐ€ ๋ฐœ์ƒํ•˜๊ธฐ ๋•Œ๋ฌธ์—
    ๋นˆ ํ•จ์ˆ˜๋ฅผ ์ถ”๊ฐ€ํ•ด์ฃผ์–ด ์—๋Ÿฌ๋ฅผ ๋ฐฉ์ง€ํ–ˆ์Šต๋‹ˆ๋‹ค
@Sangjun-man Sangjun-man merged commit 2c27b54 into qa Sep 16, 2023
@Sangjun-man Sangjun-man deleted the fix/register_next branch October 15, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant