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

kuring-98 학과 편집 화면 2.0 디자인 구현 #91

Merged
merged 36 commits into from
Feb 5, 2024

Conversation

mwy3055
Copy link
Member

@mwy3055 mwy3055 commented Jan 31, 2024

https://kuring.atlassian.net/browse/KURING-98?atlOrigin=eyJpIjoiNzk5MGYzM2YyOGViNDZiNWIwZmZmMGZhMWE5ODczMTEiLCJwIjoiaiJ9

학과 편집 화면에 2.0 디자인을 적용했습니다.

image

_2024_01_31_17_00_55_571.mp4

구현 상세

대부분 Figma 디자인대로 구현했습니다.

다만, 모두 삭제 버튼을 눌렀을 때의 화면은 디자인되어 있지 않아, 임의로 팝업을 추가했습니다. 이 부분은 디자인 팀과 논의 후 수정할 예정입니다.

image

image

DB에서 학과 구독 작업은 영문 풀네임으로 이루어진다.
@auto-assign auto-assign bot requested a review from yeon-kyu January 31, 2024 08:14
@mwy3055
Copy link
Member Author

mwy3055 commented Feb 1, 2024

으악 하나 실수했습니다 ㅠㅠ kuring-98이라고 써야 하는데 kuring-9라고 썼네요

Copy link
Contributor

@x-0o0 x-0o0 left a comment

Choose a reason for hiding this comment

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

Diff가 너무커서 나중에는 PR을 분리하먼 좋을 거 같아요.
권장드리는 diff는 2-300정도고
1천줄이 넘어가면 리뷰퀄리티가 급격히 떨어집니닷.
제가 리뷰하려면 클래스, 시퀀스 다이어그램 두개 그려봐야 파악이 될거 같아요.
연규님의 리뷰만 믿겠습니다.

@yeon-kyu
Copy link
Member

yeon-kyu commented Feb 4, 2024

으악 하나 실수했습니다 ㅠㅠ kuring-98이라고 써야 하는데 kuring-9라고 썼네요

Edit Commit Message 하시고 force-push 하시면 PR유지되면서 변경됩니다ㅎㅎ

@x-0o0
Copy link
Contributor

x-0o0 commented Feb 4, 2024

으악 하나 실수했습니다 ㅠㅠ kuring-98이라고 써야 하는데 kuring-9라고 썼네요

Edit Commit Message 하시고 force-push 하시면 PR유지되면서 변경됩니다ㅎㅎ

Or squash & merge 할거라면 스쿼시 할때 메세지 수정 하시면 됩니다.
지금 iOS 레포는 자동으로 머지시 스쿼시를 하고 있어요. (Base 브랜치에 불필요하게 많은 커밋 기록이 있을 필요는 없어서)

Copy link
Member

@yeon-kyu yeon-kyu left a comment

Choose a reason for hiding this comment

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

LGTM~!

@mwy3055 mwy3055 merged commit 814eb35 into 2.0/base Feb 5, 2024
1 check passed
@mwy3055 mwy3055 deleted the 2.0/edit_departments branch February 5, 2024 07:38
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.

3 participants