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

Modal api 적용 #28

Merged
merged 12 commits into from Jun 12, 2024
Merged

Modal api 적용 #28

merged 12 commits into from Jun 12, 2024

Conversation

ghost
Copy link

@ghost ghost commented Jun 12, 2024

타입 에러에서 시간이 생각보다 오래 걸렸습니다..
modal api 적용이 늦어져서 죄송합니다.

아래에 있는 기능 빼고는 테스트 했을 때 네트워크 응답에서 에러가 없었는데 문제가 생기면 최대한 빨리 수정하겠습니다.

  • 댓글 추가 기능
  • 댓글 수정 기능
  • 댓글 삭제 기능
  • 카드 이미지 api 적용
  • 할 일 모달 - 상태, 담당자 데이터 추가

hook을 삭제하고 파일을 수정해서 대시보드 추가, alert 모달 빼고는 사용방법이 달라졌습니다. 확인 부탁드립니다.
모달참고


수정하기 전 형태로 사용한 부분은 제가 일단 변경해두었는데 수정이 필요할 것 같습니다.

초대하기 모달
dashboard.{dashboardid} > components > Btn > Btn.tsx


제가 사용할 때 파일에서 타입 해결을 못하고 apiColumns, apiCards 에서 수정한 부분이 생겼습니다.
수정 후 에러는 없었는데 변경으로 문제되는 부분을 알려주시면 다시 수정하겠습니다.

-apiColumns
~ instance.<>('/cards', body);
부분에 경로를 cards에서 columns로 변경했습니다.

-apiCards
카드 정보를 가져와서 사용할 때 응답 타입이 안 맞다고 타입에러가 떠서 CardOverAll에 dashboardId: number;, teamId: string;를 추가했습니다.

export interface CardOverAll { id: number; title: string; description: string; tags: string[]; dueDate: string; assignee: { profileImageUrl?: string; nickname: string; id: number; }; imageUrl?: string; columnId: number; dashboardId: number; createdAt: string; updatedAt: string; teamId: string; }

@ghost ghost requested review from eunji-0623, ssumai-kr, foxholic9 and imsoohyeok June 12, 2024 02:21
Copy link
Owner

@eunji-0623 eunji-0623 left a comment

Choose a reason for hiding this comment

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

확인했습니다.!

@foxholic9 foxholic9 merged commit ff45af3 into eunji-0623:dev Jun 12, 2024
3 checks passed
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