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

링크 생성하기 뷰 링크 생성 post 누나쿵 #146

Merged
merged 10 commits into from
Jan 21, 2022
Merged

Conversation

NamJwong
Copy link
Member

⛓ Related Issues

📋 작업 내용

  • 링크 생성하기 뷰 링크 생성 post 누나쿵 - [x] ~ 페이지 구조화 및 스타일링

📌 PR Point

누나쿵

@NamJwong NamJwong added the feature 🎄 기능 개발 label Jan 21, 2022
@NamJwong NamJwong self-assigned this Jan 21, 2022
Copy link
Member

@SeojinSeojin SeojinSeojin left a 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) => {
Copy link
Member

Choose a reason for hiding this comment

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

이 부분 내 코드에 있던 부분 같은데 커밋이 된겨??

Copy link
Member

Choose a reason for hiding this comment

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

지금 dev에 있는 코드랑 아예 다른 것 같은데 머지하고 다시 푸시해줄 수 있어?

Copy link
Member Author

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]);
Copy link
Member

Choose a reason for hiding this comment

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

지워 주세요!!

Copy link
Member Author

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}`);
Copy link
Member

Choose a reason for hiding this comment

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

우리 도메인으로 바꾸어 주세요!!

링크를 state로 관리한건 아주 좋은 선택인듯!

대신 그럼 아래에서 클립보드 복사 눌렀을 때 링크가 빈 스트링("")일 경우 다시 시도하라고 말해줘야 할듯해!!

Copy link
Member Author

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);
Copy link
Member

Choose a reason for hiding this comment

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

지워 주세요!

Copy link
Member Author

Choose a reason for hiding this comment

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

지웠다리

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

Copy link
Member

@SeojinSeojin SeojinSeojin left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!!!!!!

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

@NamJwong NamJwong merged commit 0870922 into dev Jan 21, 2022
@NamJwong NamJwong deleted the feat/#103 branch January 21, 2022 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 🎄 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

너가소개서 링크 생성 api 연결하기
4 participants