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

DEP-186 ui: 짝꿍 선택에 따라 효과 추가 #58

Merged
merged 10 commits into from
Dec 7, 2022

Conversation

juhwankim-dev
Copy link
Member

@juhwankim-dev juhwankim-dev commented Dec 5, 2022

💁‍♂️ 변경 내용

KakaoTalk_20221206_010441265.mp4

AS-IS

  • 짝꿍 선택을 해도 아무런 반응이 없었습니다.

TO-BE

  • 짝꿍을 선택하면 1dp 짜리 테두리가 생깁니다.
  • 짝꿍을 선택하면 ripple 효과가 나타납니다.

📢 전달사항

테두리만 바뀌게 했더니 알아채지 못할 정도의 사소한 변화여서
재미로 ripple effect를 줘봤습니다.

rebase 했는데 또 커밋이 섞여버렸습니다 🥲

  1. Develop에서 브랜치 A를 따서 작업하고 Push 했음
  2. 리뷰를 기다리는 동안 로컬에 있는 A 브랜치에서 다음 작업을 했음
  3. 리뷰가 돼서 Develop에 Merge and Squash를 함
  4. 그 다음 로컬에서 하던 작업이 끝나 Develop에 Rebase함
  5. Push함
  6. 이전 커밋까지 push됨

이렇게 되어버렸네요
혹시 4번에서 Rebase를 하면 안되는걸까욤,,

봐야하는 커밋은 아래 3개입니다.

ui: 테두리가 있는 rect 도형 구현
feat: 짝꿍 선택 상태를 주고 받는 fragment와 viewModel 구현
chore: id 지정

@juhwankim-dev juhwankim-dev added the enhancement New feature or request label Dec 5, 2022
@juhwankim-dev juhwankim-dev self-assigned this Dec 5, 2022
@juhwankim-dev juhwankim-dev requested a review from a team as a code owner December 5, 2022 16:12
@sonarcloud
Copy link

sonarcloud bot commented Dec 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@junhaesung
Copy link
Member

A브랜치의 커밋 id 가 각각 1,2,3 이고, B브랜치의 커밋 id 가 4 라고 했을 때,
A브랜치가 develop 에 머지되면, squash merge 방식이니까 1,2,3 커밋이 합쳐진 5 커밋이 develop 에 새로 생길거에요
B입장에서 develop 에 rebase 한다면, 1,2,3, 5, 4 가 되는거같네요..

develop 머지 후에 C 브랜치를 만들고, 4 커밋을 cherry-pick 하면 될거같습니당
더 편한 방법이 있는지는 알아볼게요!

@juhwankim-dev
Copy link
Member Author

A브랜치의 커밋 id 가 각각 1,2,3 이고, B브랜치의 커밋 id 가 4 라고 했을 때, A브랜치가 develop 에 머지되면, squash merge 방식이니까 1,2,3 커밋이 합쳐진 5 커밋이 develop 에 새로 생길거에요 B입장에서 develop 에 rebase 한다면, 1,2,3, 5, 4 가 되는거같네요..

develop 머지 후에 C 브랜치를 만들고, 4 커밋을 cherry-pick 하면 될거같습니당 더 편한 방법이 있는지는 알아볼게요!

아하..! Squash and merge라서 그런거군요
이렇게 또 알아갑니다. 감사합니다!

@juhwankim-dev juhwankim-dev merged commit 90b3d4d into develop Dec 7, 2022
@juhwankim-dev juhwankim-dev deleted the feature/DEP-186_create_mate_step_2_ui branch December 10, 2022 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants