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

결과 로직 알고리즘 짜기 #371

Merged
merged 5 commits into from
Jul 9, 2023
Merged

Conversation

sumi-0011
Copy link
Member

@sumi-0011 sumi-0011 commented Jul 8, 2023

🤔 해결하려는 문제가 무엇인가요?

결과 로직 알고리즘 짜기

🎉 변경 사항

  • util 폴더에 성향 결과 그룹을 도출하는 알고리즘을 작성했어요.

🙏 여기는 꼭 봐주세요!

  • 파일 이름이나, 네이밍을 고민했는데 일단 알고리즘에 집중해서 만들었어요. 수정할 부분이 있다면 말해주세요!
  • src/hooks/lifeCycle/useIsMounted.ts 이거는 제가 작성한건 아닌데, 처음에 main이 base 브랜치였어서 추가된 것 같아요!

사용 방법

 // 결과 그룹 도출
 const resultGroup = getResultGroup(tendencyCountData);

@sumi-0011 sumi-0011 changed the base branch from main to feature/result/base July 8, 2023 10:40
@github-actions github-actions bot added the test: unit Generate or update unit test label Jul 8, 2023
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (eaa0f5d) 91.00% compared to head (a54961b) 91.00%.

Additional details and impacted files
@@                 Coverage Diff                  @@
##           feature/result/base     #371   +/-   ##
====================================================
  Coverage                91.00%   91.00%           
====================================================
  Files                       43       43           
  Lines                      300      300           
  Branches                    52       52           
====================================================
  Hits                       273      273           
  Misses                      27       27           
Impacted Files Coverage Δ
src/hooks/lifeCycle/useIsMounted.ts 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link

github-actions bot commented Jul 8, 2023

Bundle Sizes

Compared against 8e54cfe

Route Size (gzipped) Diff
/result/dna 86.66 KB added

Dynamic import: No significant changes found

Copy link
Member

@hyesungoh hyesungoh left a comment

Choose a reason for hiding this comment

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

👍 👍 고생하셨어요 ~~ 👍 👍 👍

나중에 테스트 코드를 넣어도 좋을 부분이라 생각되네요 굿굿~~

@hyesungoh hyesungoh merged commit 29ffa68 into feature/result/base Jul 9, 2023
14 checks passed
@hyesungoh hyesungoh deleted the feat/result-logic branch July 9, 2023 05:09
@sumi-0011 sumi-0011 linked an issue Jul 11, 2023 that may be closed by this pull request
hyesungoh pushed a commit that referenced this pull request Jul 14, 2023
* feat: useIsMounted 작성 (#71)

* feat: 결과 로직 짜기 - 많이 선택된 성향 찾기

* feat: 결과 로직 카운트 수정

* feat: 결과 그룹 도출

* feat: 결과 그룹 도출 로직 연결

---------

Co-authored-by: Sangjo LEE <cent7425@gmail.com>
sumi-0011 added a commit that referenced this pull request Jul 20, 2023
* result DNA 페이지 작업 중 (#357)

Co-authored-by: 권오연 <ohyeon.kwon@flask.global>

* feat: 나와 잘맞는 DNA 배너 (#366)

* feat: 나와 잘맞는 DNA 배너

* Update src/components/banner/DnaBanner.tsx

Co-authored-by: hyesung oh <haesungoh414@gmail.com>

---------

Co-authored-by: oyeon-kwon <ohyeon.kwon@flask.global>
Co-authored-by: hyesung oh <haesungoh414@gmail.com>

* 결과 로직 알고리즘 짜기  (#371)

* feat: useIsMounted 작성 (#71)

* feat: 결과 로직 짜기 - 많이 선택된 성향 찾기

* feat: 결과 로직 카운트 수정

* feat: 결과 그룹 도출

* feat: 결과 그룹 도출 로직 연결

---------

Co-authored-by: Sangjo LEE <cent7425@gmail.com>

* DNA 페이지 API 관련 로직 (#365)

* feat: use get user info by survey id

* feat: use get tendency와 로딩 핸들링

* feat: dna 페이지 접근 유저 확인

* 결과 페이지 상단 DNA 배너 (#370)

* feat: 결과 페이지 dna 배너 피드백 없을 때

* feat: 결과 페이지 상단 배너

* feat: 북마크 아이콘 피드백 컴포넌트에 추가 (#369)

* feat: useIsMounted 작성 (#71)

* feat: 북마크 아이콘 피드백 컴포넌트에 추가

* fix: isView Props 삭제

---------

Co-authored-by: Sangjo LEE <cent7425@gmail.com>
Co-authored-by: oyeon-kwon <ohyeon.kwon@flask.global>

* dna 페이지 퍼블리싱 및 로직 추가 (#372)

* feat: 성향 데이터

* feat: 결과 dna 페이지 알고리즘 사용

* feat: dna 페이지 배너 사용 및 결과 렌더링

* feat: 외부자 dna 페이지 대응

* DNA 결과 로직 테스트 코드 작성 (#373)

* feat: 성향 데이터

* feat: 결과 dna 페이지 알고리즘 사용

* feat: dna 페이지 배너 사용 및 결과 렌더링

* feat: 외부자 dna 페이지 대응

* fix: 타입 선언 충돌 수정

* test: 결과 로직 테스트 코드 작성

* fix: console 제거

* fix: build error type fix

* refactor: 주석 제거

---------

Co-authored-by: hyesungoh <haesungoh414@gmail.com>

* 서버에서 SVG DNA 이미지 만들기 (#381)

* feat: 성향 데이터

* feat: 결과 dna 페이지 알고리즘 사용

* feat: dna 페이지 배너 사용 및 결과 렌더링

* feat: 외부자 dna 페이지 대응

* chore: satori 설치

* feat: image server에서 만들기

* feat: 강제 html 집어넣기

* feat: 일단 귀여운 이미지 배치

* refactor: 안쓰는거 정리

* feat: text 배치

* fix: 안쓰는 모듈 제거

* refactor: prod base url 추가 (될지는 모르겠음)

* feat: create image 정리

* fix: build error fix

---------

Co-authored-by: hyesungoh <haesungoh414@gmail.com>

* satori로 만든 svg를 png로 변환 (#382)

* feat: 성향 데이터

* feat: 결과 dna 페이지 알고리즘 사용

* feat: dna 페이지 배너 사용 및 결과 렌더링

* feat: 외부자 dna 페이지 대응

* chore: satori 설치

* feat: image server에서 만들기

* feat: 강제 html 집어넣기

* feat: 일단 귀여운 이미지 배치

* refactor: 안쓰는거 정리

* feat: text 배치

* fix: 안쓰는 모듈 제거

* refactor: prod base url 추가 (될지는 모르겠음)

* feat: create image 정리

* fix: build error fix

* chore: resvg 라이브러리 설치

* feat: resvg로 png base encoding

---------

Co-authored-by: hyesungoh <haesungoh414@gmail.com>

* refactor: 페이지 경로 변경

* feat: 이미지 다운로드 기능 추가

* feat: 모바일 감지해서 토스트 메시지 띄우기

* feat: mock data 추가

* feat: 폰트 적용

* feat: 화질 깨지는 문제 수정

* feat: 포지션  집어넣기

* feat: 배너 완성

* fix: minor style fix

* feat: 이미지 변경

* feat: 포지션 추가

* refactor: 이미지 로직 분리

* feat: pc 다운

* feat: 다운로드 추가

* fix: 충돌 에러 수정

* feat: image 경로 변경

* refactor: 사용하지 않는 파일 삭제

---------

Co-authored-by: 권오연 <61301574+oyeon-kwon@users.noreply.github.com>
Co-authored-by: 권오연 <ohyeon.kwon@flask.global>
Co-authored-by: hyesung oh <haesungoh414@gmail.com>
Co-authored-by: Sangjo LEE <cent7425@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test: unit Generate or update unit test
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

결과 로직 짜기
4 participants