feat : 20241012_API-cancelled-문제-해결하기 #95
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Comment on lines
22
to
24
| <img src="/Users/hayeon/Downloads/마이네임안바뀜.gif" alt="이름안바뀜"> | ||
|
|
||
| <img alt="cancelled" src=""/> |
There was a problem hiding this comment.
Suggested change
| <img src="/Users/hayeon/Downloads/마이네임안바뀜.gif" alt="이름안바뀜"> | |
| <img alt="cancelled" src=""/> | |
| <img src="/Users/hayeon/Downloads/마이네임안바뀜.gif" alt="이름안바뀜"> | |
| <img alt="cancelled" src=""/> |
실수하신거같아서 요거 수정이 필요할거같아요~
Comment on lines
102
to
103
| '과연 여기서 api요청에 가해질 수 있는 외부압력은 어떤 종류의 것일까?' 조금 더 원초적인 코드 로직이 무엇인가를 살펴보면, 해당 button을 누른다는 것은 form 태그에 종속된 input의 내용을 보내는 동작을 수행하는 것입니다. | ||
| 그런데 form 태그란 어떤 동작을 하고 있을까요? form이라는 태그는 button이 눌리면 브라우저를 새로고침하면서 input의 내용을 전송하려고 합니다. |
There was a problem hiding this comment.
"새로고침"이라는 용어는 잘못된거같아요. <form> 태그에 정의된 action을 수행하는 것이라고 표현하는것이 맞습니다.
godhyzzang
requested review from
day1companyhaewooklee and
day1chan99k
as code owners
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.