-
Notifications
You must be signed in to change notification settings - Fork 5
[FEAT] 외부이슈 작성/상세조회/수정 API 연동 #192
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
Conversation
… 처리 후 ExternalDetail에 API 연동 완료
sunhwaaRj
left a comment
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.
깃허브와 슬랙 연동되는 모습까지 보여주셔서 감사합니다. 데모데이 이전까지 하기 어려운 요소 있다면 말씀해주세욧
waldls
left a comment
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.
확인했습니다 수고하셨어요! 머지 이후에 백엔드 측에 경로 수정 요청해주시면 될 것 같습니다.
| content: string; | ||
| priority: string; | ||
| state: string; | ||
| goalId?: Pick<Goal, 'id'>; // TODO: 데이터 구조 통일해달라고 하기 |
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.
아 이거는 아직 안 된 건데 데모데이 이후에 반영해달라고 말씀드리려고 합니다!
체크리스트
📌 관련 이슈번호
✅ Key Changes
개별 팀과 워크스페이스 기본 팀의 외부이슈 작성/상세조회/수정 API를 연동했습니다.
external.ts에 타입 구조 정의usePostCreateExternalDetail.ts)useGetExternalDetail.ts)usePatchExternalDetail.ts)ExternalDetail.tsx,WorkspaceExternalDetail.tsx컴포넌트에 API 연동📸 스크린샷 or 실행영상
실행 영상 대신 작업하면서 찍은 스크린샷을 아래와 같이 첨부합니다.
⬇️ 깃허브 연동 이슈 작성 시 - 해당 깃허브 레포에 동일 이슈 등록된 모습


⬇️ 슬랙 연동 이슈 작성 시 - 해당 슬랙 채널에 동일 이슈 등록된 모습

💬 To Reviewers
로컬에서 깃허브<->외부이슈 API 연동이 제대로 되는지 테스트하기 위해, 일시적으로 백엔드 측에서 Veco 깃허브 앱 설정을 개발 서버로 연결해두셨습니다.
이 pr이 머지되고 나면 백엔드 측에 다시 깃허브 앱 설정을 배포 서버로 연결해두시도록 요청 드릴 예정입니다. 해당 연결이 완료된 후에, 배포환경에서 외부이슈 연동이 정상적으로 가능할 예정이니 참고 부탁드립니다. @vecosystem/veco_frontend
외부이슈 api 연동이 많이 늦었네요. 이후 금일내로 QA 수정사항 반영 모두 진행할 예정입니다.......