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

update #2626

Closed
Closed

Conversation

kkongnamul11
Copy link
Contributor

isNotFor 주석 해제

@Tnks2U
Copy link
Collaborator

Tnks2U commented Dec 21, 2023

@kkongnamul11
#2621 PR과 동일한 내용으로 보이는데요. 어느쪽을 merge하면 될까요?

@kkongnamul11
Copy link
Contributor Author

@Tnks2U
#2626 로 Merge 부탁드립니다. 해당 부분이 isnotFor 부분 추가 된 내용입니다.

@Tnks2U
Copy link
Collaborator

Tnks2U commented Dec 23, 2023

@kkongnamul11
네 알겠습니다. 나머지 PR은 close 부탁드립니다.

@Tnks2U
Copy link
Collaborator

Tnks2U commented Dec 23, 2023

@kkongnamul11
추가로 샘플제공에 동의주셨는데, 배송하셨는지 여부를 알려주시면 감사하겠습니다.

@Tnks2U
Copy link
Collaborator

Tnks2U commented Dec 23, 2023

@kkongnamul11
타깃 브랜치가 develop으로 되어 있어서 머지가 불가능합니다. develop-hw로 변경 부탁드립니다.

@kkongnamul11
Copy link
Contributor Author

@Tnks2U 혹시 다시 Commit후 재PR해야 하는건가요?

@kkongnamul11
Copy link
Contributor Author

@kkongnamul11 추가로 샘플제공에 동의주셨는데, 배송하셨는지 여부를 알려주시면 감사하겠습니다.

@Tnks2U 혹시 샘플 발송 위치좀 알려주실 수 있나요?

@Tnks2U
Copy link
Collaborator

Tnks2U commented Dec 26, 2023

@kkongnamul11
스크린샷 2023-12-26 오후 3 45 28
edit 버튼을 누르시면 브랜치 변경이 가능합니다. develop-hw로 수정 부탁드립니다. 금일내로 미수정시 이번 배포에서 반영이 힘들 수 있습니다.

@kkongnamul11 kkongnamul11 changed the base branch from develop to develop-hw December 26, 2023 06:58
@Tnks2U
Copy link
Collaborator

Tnks2U commented Dec 27, 2023

@kkongnamul11
entryjs쪽 id의 양식이 맞지 않아서 연결이 되지않아 수정하였습니다.
https://github.com/entrylabs/entryjs/pull/2631/files
가급적 타사코드를 참고하셔서 양식확인을 부탁드립니다.

@Tnks2U Tnks2U closed this Jan 18, 2024
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.

2 participants