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

TicketDetail route 변경 #292

Closed
HamBP opened this issue Sep 18, 2024 · 0 comments · Fixed by #296
Closed

TicketDetail route 변경 #292

HamBP opened this issue Sep 18, 2024 · 0 comments · Fixed by #296
Assignees
Labels
bug 버그 픽스

Comments

@HamBP
Copy link
Member

HamBP commented Sep 18, 2024

image

내비게이션 라이브러리 버전을 2.8.x 로 마이그레이션 하는 과정에서 발견.

  • TicketDetail을 navigation으로 래핑하는 과정에서 기존 TicketDetail의 argument를 제거하고 navigation의 argument로 추가
  • 기존의 TicketDetail의 route에서는 argument 정보를 제거해 주었으나 argument는 남아 있는 상황
  • route와 arguments 정보 불일치에 대한 유효성 검사가 2.8.x 버전에서 추가된 것으로 추정

관련 PR #261

image

route에는 arguements가 없으나 arguments를 지정한 상황.
arguments만 제거하면 해결될 거 같긴 한데, 래핑한 navigation이랑 TicketDetail이 사용하는 route를 아예 다르게 가져가면 어떨까 싶어

image

@HamBP HamBP added the bug 버그 픽스 label Sep 18, 2024
@HamBP HamBP self-assigned this Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 버그 픽스
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant