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 2 commits into from
Feb 21, 2024

Conversation

changhwan77
Copy link
Collaborator

@changhwan77 changhwan77 commented Feb 18, 2024

Title

✨ 판매 페이지 레이아웃 및 로그아웃 시 홈화면 로그인 버튼 보이도록 수정

What type of PR is this?

  • 🍕 Feature ( 새로운 기능 추가 )
  • 🐛 Bug Fix ( 버그 수정 )
  • 📝 Documentation Update ( 개발자 관련 문서 업데이트 )
  • 🎨 Style ( 코드 포맷팅, 세미콜론 누락, 코드 변경이 없는 경우 )
  • 🧑‍💻 Code Refactor ( 코드 리펙토링 )
  • 🔥 Performance Improvements ( 성능 향상 )
  • ✅ Test ( 테스트 코드, 리펙토링 테스트 코드 추가 )
  • 🤖 Build ( 빌드 )
  • 🔁 CI
  • 📦 Chore (Release) (빌드 업무 수정, 패키지 매니저 수정 )
  • ⏩ Revert ( 커밋을 날린 경우 )

Description

  1. 판매 페이지 레이아웃 및 상품 카드&배너를 추가했어요.
  • as-is: 로그아웃 시 트티 홈화면에 상태가 반영이 안돼서 로그인 버튼이 안뜸
  • to-be: 로그아웃 상태를 공유해 홈화면에서 로그인 버튼이 뜨도록 수정

Related Issue number and link

  • Jira :
  • Issue :

Mobile & Desktop Screenshots/Recordings

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

To Reviewers

ConstraintedScaffold 위젯의 경우 width가 700.w으로 제한된 컨테이너에 child를 넣는거라 먼저 자식 위젯을 어떻게 얼라인할 것인지(Row | Column)를 정한 후에 위젯을 배정해야 layout이 정상 작동되는 것을 알았어요!

추후에 혹시나 사용할 일이 있다면 참고하시면 되겠습니다.

@changhwan77 changhwan77 merged commit daf1e1c into master Feb 21, 2024
1 check failed
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.

1 participant