Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

편집 취소버튼 눌러도 저장 #103

Closed
wants to merge 1 commit into from
Closed

Conversation

soyekwon
Copy link
Contributor

버그라 하기도 애매한... 기능 추가가 맞는 듯

@soyekwon soyekwon added the 🐛 Bug Something isn't working label Feb 23, 2022
@soyekwon soyekwon self-assigned this Feb 23, 2022
@netlify
Copy link

netlify bot commented Feb 23, 2022

✔️ Deploy Preview for todaytheylearned ready!

🔨 Explore the source changes: 183acc2

🔍 Inspect the deploy log: https://app.netlify.com/sites/todaytheylearned/deploys/62160cd36760c100081a43da

😎 Browse the preview: https://deploy-preview-103--todaytheylearned.netlify.app/

@soyekwon soyekwon linked an issue Feb 23, 2022 that may be closed by this pull request
@soyekwon soyekwon changed the title #101 편집 취소버튼 눌러도 저장 Feb 23, 2022
@minjj0905
Copy link
Member

pr은 프로젝트 연결은 안해도 된답니단

@minjj0905 minjj0905 added the 👀 In review 리뷰 요청중인 상태 label Feb 23, 2022
@minjj0905
Copy link
Member

base브랜치 바꿔야해요 ㅠㅠ 지금 main 브랜치에 병합중이어서 제가 develop으로 고쳐놓음..

@minjj0905 minjj0905 changed the base branch from main to develop February 23, 2022 10:47
@@ -107,7 +107,8 @@ const ProfileEdit = () => {
/>
</Styled.AvatarFormContainer>
<Styled.ActionContainer>
<Styled.BackButton>취소</Styled.BackButton>
<Styled.BackButton onClick={props.onChangeMode}>취소</Styled.BackButton>

<Styled.EditButton>저장</Styled.EditButton>
Copy link
Member

Choose a reason for hiding this comment

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

p2: 저장버튼은 form submit 용이면 option에 type="submit" 이런걸 넣어주는게 좋습니다
back button은 반대로 form submit 을 안하니까 type="button" 으로 지정해줍시다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

댑악 넹

Copy link
Member

@cychann cychann left a comment

Choose a reason for hiding this comment

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

확인했습니다 👍

@minjj0905 minjj0905 added 🛠 Developing pr을 생성했지만, 리뷰사항 반영 혹은 버그 수정등의 이유로 해당 브랜치를 수정중일 경우 🙅 Duplicate 중복된 커밋이 있을 경우 and removed 👀 In review 리뷰 요청중인 상태 🛠 Developing pr을 생성했지만, 리뷰사항 반영 혹은 버그 수정등의 이유로 해당 브랜치를 수정중일 경우 labels Feb 23, 2022
@minjj0905 minjj0905 closed this Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working 🙅 Duplicate 중복된 커밋이 있을 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

프로필 편집 취소 버튼 눌러도 저장
3 participants