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

[SP3/refactor] maindashboard orkTree 동작 오류 해결 #327

Open
wants to merge 16 commits into
base: SP3/refactor
Choose a base branch
from

Conversation

SooY2
Copy link
Member

@SooY2 SooY2 commented Oct 20, 2024

🔥 Related Issues

💜 작업 내용

  • task 추가 onBlur
  • edit 데이터 패칭 이슈 핸들링
  • krList 3개 다 채워도 + 버튼 안없어지는 이슈 핸들링

✅ PR Point

  • edit 에서 수정하고 view로 넘어가면 바로 반영이 안되는 이슈
    view모드에서 보여지는 OKRTree 데이터를 edit모드에서 보여주는 데이터로 변경했습니당

++ 동작 오류 발견하면 말해쥬세요!!

☀️ 스크린샷 / GIF / 화면 녹화

2024-10-20.20.13.12.mov

📚 Reference

eonseok-jeon and others added 16 commits May 31, 2024 22:42
* feat: GA 이벤트 탐지를 위한 class name 추가

* chore: class name 수정

* feat: kr/task 삭제에 따른 class name 분기 처리

* feat: 가이드/직접 선택에 따른 class name 분기 처리
* feat : social get fetcher 생성

* feat : social data type 추가

* fix : social 데이터 목 데이터 -> 실제 데이터 연결로 수정

* fix : 안쓰게 된 mock data 파일 삭제
* [Refactor] placeholder 라이팅 수정 (#309)

* fix : kr objective placeholder 개선

* fix : 가이드에 따라 설정 시 kr 목표 텍스트 변경

* fix : kr 단위 힌트 메시지 수정

* fix : 가이드에 따라 설정 kr placeholder 재수정

* fix : objContent placeholder 수정

* [Refactor] KR validation 로직 추가 (#310)

* fix : krValidation 로직에 krTitle, krMetric 문자만 입력 추가

* fix : 가이드에 따라 설정 - 첫번째 kr 카드 krValid 함수 사용으로 수정
* fix: nickname regex 수정
* chore: 워딩 수정

* chore: 워딩 수정
* chore: 워딩 수정

* fix: 버튼 링크 수정
@SooY2 SooY2 self-assigned this Oct 20, 2024
Copy link

vercel bot commented Oct 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
moonshotyou ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 20, 2024 11:17am

@SooY2 SooY2 changed the title Refac/#323/main dashboard okr [SP3/refactor] maindashboard orkTree 동작 오류 해결 Oct 20, 2024
@Yeonseo-Jo
Copy link
Contributor

Yeonseo-Jo commented Oct 20, 2024

커밋이 섞인것 같은데 base 브랜치 기준으로 rebase 쳐서 함 정리해주면 리뷰하기 더 편할것 같습니댜 !!

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.

3 participants