-
Notifications
You must be signed in to change notification settings - Fork 1
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
링크 생성하기 뷰 링크 생성 post 누나쿵 #146
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!!!!
return NEOSOSEO_FORM_DATA.FORM; | ||
}; | ||
|
||
const postFormAnswer = async (body: NeososeoAnswerData) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분 내 코드에 있던 부분 같은데 커밋이 된겨??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지금 dev에 있는 코드랑 아예 다른 것 같은데 머지하고 다시 푸시해줄 수 있어?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해결 완뇨
}, []); | ||
|
||
useEffect(() => console.log('here'), [link]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지워 주세요!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어머 알겟슘다 ,,
const q = await api.neososeoFormService.postCreateForm(Number(formID), () => | ||
navigate(`/neoga/create/${formID}/created`), | ||
); | ||
setLink(`http://localhost:3000/neososeoform/${q}`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
우리 도메인으로 바꾸어 주세요!!
링크를 state로 관리한건 아주 좋은 선택인듯!
대신 그럼 아래에서 클립보드 복사 눌렀을 때 링크가 빈 스트링("")일 경우 다시 시도하라고 말해줘야 할듯해!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정 완뇨
console.log(e.response); | ||
}); | ||
if (response.status === 200 && response.message === '폼 생성 성공') { | ||
console.log(response.data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지워 주세요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지웠다리
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!!!!!!!
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~ |
⛓ Related Issues
📋 작업 내용
📌 PR Point
누나쿵