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

🔀 :: (#358) - 프로필 세부스택 삭제 기능 구현 #360

Merged
merged 5 commits into from
Sep 4, 2023

Conversation

khs3994
Copy link
Contributor

@khs3994 khs3994 commented Sep 4, 2023

💡 개요

  • 프로필 세부스택 삭제 기능 구현

🔀 변경사항

  • 프로필 테크스택은 profileDetailTechStack이라는 변수를 참조하고 있지만 실제로는 ProfileData만 copy를 통해 지워주기 때문에 UI상에서 지워지지 않던 문제를 profileDetailTechStack도 삭제 버튼 클릭시 함께 값을 없애주도록 변경하여 문제를 해결함

🖥️ 주요코드

onTechStackRemoved = { removedItem ->
    onTechStackItemRemoved(removedItem)
    onProfileValueChanged(data.copy(techStack = detailStacks.minus(removedItem)))
}

@khs3994 khs3994 added 0️⃣ Priority: Critical 우선순위 - 긴급!!!!! ⚡️ Simple 간단한 변경사항 ✨ Feature 신규 기능 labels Sep 4, 2023
@khs3994 khs3994 requested a review from leehyeonbin September 4, 2023 02:59
@khs3994 khs3994 self-assigned this Sep 4, 2023
@khs3994 khs3994 linked an issue Sep 4, 2023 that may be closed by this pull request
@khs3994 khs3994 merged commit 23a4a95 into develop Sep 4, 2023
@khs3994 khs3994 deleted the feature/358_make_profile_detail_stack_delete branch September 4, 2023 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0️⃣ Priority: Critical 우선순위 - 긴급!!!!! ✨ Feature 신규 기능 ⚡️ Simple 간단한 변경사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

프로필 세부스택 삭제
2 participants