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

고라우터 추가 #10

Merged
merged 5 commits into from
Jul 25, 2024
Merged

Conversation

jwson-automation
Copy link
Contributor

설명

고 라우터 추가

변경 사항

  • main.dart에서 Go_router 프로바이더를 처음부터 전체 프로젝트에 적용.
  • Navigator가 아닌 context.go를 이용해서 화면 이동이 가능해졌습니다.
  • Web에서 일부 화면 전환해서 LayoutBuilder가 추가되지 않던 문제를 하나의 router에서 이동을 시키는 것을 이용해서 해결했습니다.

체크리스트

< 체크리스트 항목을 확인해 주세요. >

  • 오늘도 행복하게 코딩했는가?
  • release 브랜치를 제대로 최신화 하고 이 브랜치에 merge 했는가?
  • PR 제목은 명확하고 간결한가?
  • PR 에는 하나의 작업에 대한 내용만 포함되었는가?
  • 파일명은 누구나 이해할 수 있게 작성되었는가?
  • camelCase를 사용하였는가? (ThisIsCamelCase, this_is_not_camel_case)
  • 텍스트는 AppStrings.dart 파일에서 가져오고 있는가?
  • 디스코드 채팅방에 완료된 태스크를 알렸는가?

스크린샷

< 여기에 이미지/동영상을 드래그 앤 드랍하면 링크가 생성됩니다. >
sample

참조

관련된 이슈나 PR이 있다면 링크를 추가해주세요.

@jwson-automation
Copy link
Contributor Author

로그인 상태에 따라서 다르게 보여주는 기능이 필요합니다.

@jwson-automation
Copy link
Contributor Author

고민중데스

Copy link
Contributor

@hongmono hongmono left a comment

Choose a reason for hiding this comment

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

굿

@jwson-automation jwson-automation merged commit 5158e74 into release/2024-summer Jul 25, 2024
1 check passed
@jwson-automation jwson-automation deleted the feature/go_router branch July 25, 2024 13:27
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