Skip to content

Fix/register next - #2

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

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

Conversation

@Sangjun-man

Copy link
Copy Markdown
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