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

9주차미션-제이 #19

Merged
merged 13 commits into from
Dec 2, 2024
Merged

9주차미션-제이 #19

merged 13 commits into from
Dec 2, 2024

Conversation

OrangeKim04
Copy link
Collaborator

Merge pull request는 모든 리뷰가 완료된 후에 눌러주세요!

1. Add a title

  • N주차미션 - 닉네임으로 수정해 주세요! ex) 1주차 미션- 테비

2. Reviewers

  • Reviewers(리뷰어)에 팀원을 등록해 주세요! (최소 1명)

3. Assignees

  • Assignees(PR 담당자)에 본인을 등록해 주세요!

4. Labels

  • Labels에 mission 태그를 등록해 주세요!

reducers:{
// 음반수량 증가
increase: (state, action) => {
state.items = state.items.map(e => e.id === action.payload ? {...e, amount: e.amount+1}: e);
Copy link
Collaborator

Choose a reason for hiding this comment

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

전개연산자로 복사를 한 뒤에 변경사항만 변경하는 방식이 굉장히 좋은 것 같아요!

@OrangeKim04 OrangeKim04 merged commit ff10f6a into main Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants