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

🎉 제안 수락, 거절 API 연결 및 기타 작업 #92

Merged
merged 5 commits into from
Nov 21, 2023

Conversation

doggopawer
Copy link
Contributor

- 목적

관련 티켓 번호: 262


- 주요 변경 사항

  • 제안 수락/거절 기능 실제 API 적용
  • 내 물건 카드 버튼 라우팅 변경
  • MaxWidthWrapper 스타일 변경
  • 내 제안 확인 페이지, 카드 설명 섹션 생성 및 추가

기타 사항 (선택)


- 스크린샷 (선택)

@doggopawer doggopawer added the D-0 label Nov 21, 2023
@doggopawer doggopawer self-assigned this Nov 21, 2023
@doggopawer doggopawer changed the title Nabi 262 🎉 제안 수락, 거절 API 연결 및 기타 작업 Nov 21, 2023
Copy link
Collaborator

@oaoong oaoong left a comment

Choose a reason for hiding this comment

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

좋습니다! 적절하지 않은 곳에 queryString으로 라우팅하는 부분은 조금 수정이 필요할 것 같긴한데 중요사항은 아니라 리팩토링해도 충분할 것 같습니다

@doggopawer
Copy link
Contributor Author

좋습니다! 적절하지 않은 곳에 queryString으로 라우팅하는 부분은 조금 수정이 필요할 것 같긴한데 중요사항은 아니라 리팩토링해도 충분할 것 같습니다

구체적으로 어떤 곳이죠? 일단 알려주시면 todoList에 적어넣도록 하겠습니다.

@doggopawer doggopawer merged commit 541f529 into develop Nov 21, 2023
2 checks passed
@doggopawer doggopawer deleted the NABI-262 branch November 21, 2023 15:39
@oaoong
Copy link
Collaborator

oaoong commented Nov 22, 2023

좋습니다! 적절하지 않은 곳에 queryString으로 라우팅하는 부분은 조금 수정이 필요할 것 같긴한데 중요사항은 아니라 리팩토링해도 충분할 것 같습니다

구체적으로 어떤 곳이죠? 일단 알려주시면 todoList에 적어넣도록 하겠습니다.

/mypage/suggest-checks?cardId= 개인적으로 이 부분이 조금 적절하지 않다고 생각하긴 하는데 크게 중요하진 않은 것 같습니다

@oaoong
Copy link
Collaborator

oaoong commented Nov 22, 2023

그리고 페이지 대부분이 mypage/ 하위로 종속될 필요도 없을 것 같아요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants