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

[27기 길도연] 정렬 알고리즘 수정 #52

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sanha-kil
Copy link
Contributor

:: 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

  • 기능 추가
  • 리뷰 반영
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표 (해당 브랜치에서 구현하고자 하는 하나의 목표를 설정합니다.)

  • 정렬 알고리즘 수정

:: 구현 사항 설명 (작업한 내용을 상세하게 기록합니다.)

  1. 정렬 알고리즘 수정
  • 제품 리스트 정렬 알고리즘이 정상적으로 작동하지 않던 문제를 해결하였습니다.

:: 성장 포인트 (해당 기능을 구현하며 고민했던 사항이나 새로 알게된 부분, 어려웠던 점 등을 작성합니다.)

  • 객체가 담겨있는 배열에 대한 정렬에 대해 알게 되었고 정렬된 배열을 map으로 렌더링 할 때 어떻게 렌더링 되는지에 대해 고민하였습니다.

:: 기타 질문 및 특이 사항

@@ -19,6 +19,8 @@ const SortedProducts = () => {
setSelectedProduct(product);
};

console.log(productsList);
Copy link

Choose a reason for hiding this comment

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

[ESLint] Unexpected console statement. (view)

Rule Severity Category Recommended
no-console warn Possible Errors No

References:

You can close this issue if no need to fix it. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant