Skip to content

[REFACTOR/#220] 공고 상세 뷰 / 리팩토링#230

Merged
boiledeggg merged 15 commits intodevelopfrom
refactor/#220-announcement
Sep 9, 2024
Merged

[REFACTOR/#220] 공고 상세 뷰 / 리팩토링#230
boiledeggg merged 15 commits intodevelopfrom
refactor/#220-announcement

Conversation

@arinming
Copy link
Copy Markdown
Contributor

@arinming arinming commented Sep 9, 2024

⛳️ Work Description

  • 매퍼 생성
  • 네비게이터 변경
  • 다이얼로그 로직 수정

📸 Screenshot

220.mp4

📢 To Reviewers

  • Scaffold를 삭제하고 Column으로 변경하면서 바텀바가 적용이 뭔가 어색해지는데 더 연구해보겠습니다 ㅜㅜ,.,. 다이얼로그도 연결은 했는데, 스크랩 추가시 토큰을 재발급하면서 401 에러가 발생하고 추가가 되지 않아서 해당 안건 문의 드립니다 🥹🥹🥹🥹🥹🥹

@arinming arinming added UI 💐 UI 작업 아린💛 아린 labels Sep 9, 2024
@arinming arinming added this to the 2차 스프린트 작업 milestone Sep 9, 2024
@arinming arinming self-assigned this Sep 9, 2024
@arinming arinming linked an issue Sep 9, 2024 that may be closed by this pull request
4 tasks
Copy link
Copy Markdown
Member

@boiledeggg boiledeggg left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~
스크랩 문제는 서버에서 현재 색상값을 문자열로 받도록 API를 수정했는데 그걸 반영하지 못해 생긴 것 같습니다,,
최대한 빨리 반영하고 PR 올리테니 그거 먼저 머지하고 여기에 반영해서 확인하고 다시 리뷰 요청 부탁드려요!!

@boiledeggg
Copy link
Copy Markdown
Member

그리고 메인 검색 화면의 탑바 좌측에 로고는 왜 초록색 박스로 나오는 건가요??

@arinming
Copy link
Copy Markdown
Contributor Author

arinming commented Sep 9, 2024

그리고 메인 검색 화면의 탑바 좌측에 로고는 왜 초록색 박스로 나오는 건가요??

스크랩 문제 확인했습니다!
메인 검색 화면은 패딩value 적용 아직 안된 상태라 따로 이슈 파서 수정할 예정입니당

Copy link
Copy Markdown
Member

@leeeyubin leeeyubin left a comment

Choose a reason for hiding this comment

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

확인했습니당~!! 리팩 넘넘 좋아용!!!!

Comment on lines +165 to +168
internNavGraph(
paddingValues = paddingValues,
navHostController = navigator.navController
)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

적용 좋아요!!!

@boiledeggg boiledeggg merged commit c326c31 into develop Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI 💐 UI 작업 아린💛 아린

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 공고 상세 뷰 / 리팩토링

3 participants