Skip to content

[Fix]: QA 반영 - 하위 목표 뷰#213

Merged
shinjigu merged 3 commits intodevelopfrom
fix/#211/lowerTodo
Sep 25, 2025
Merged

[Fix]: QA 반영 - 하위 목표 뷰#213
shinjigu merged 3 commits intodevelopfrom
fix/#211/lowerTodo

Conversation

@shinjigu
Copy link
Copy Markdown
Contributor

💡 Summary

close #211

✅ Tasks

  • 목표 작성 없이 주기 선택 시 불필요한 API 요청 제거 -> 목표가 작성된 상태에서만 주기 변경 시 API 요청이 발생하도록 조건을 추가했습니다.
  • 할 일 8개 필드 모두 채워졌을 때 AI 버튼 비활성화 로직 추가 -> 현재 선택된 상위 목표의 하위 목표가 모두 채워졌는지 확인하여 AI 버튼 비활성화 로직을 개선했습니다.

👀 To Reviewer

다음의 동작을 확인해 주시면 감사하겠습니다 !!

  • 목표 작성 없이 주기만 선택 → API 요청 없음
  • 목표 작성 후 주기 변경 → 수정 API 호출
  • 8개의 할 일 필드 모두 채우거나 AI 사용 후 → AI 버튼 비활성화 (두 가지 경우 모두에서 AI 버튼이 비활성화되는지)

@vercel
Copy link
Copy Markdown

vercel bot commented Sep 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ninedot Ready Ready Preview Comment Sep 24, 2025 11:48am

@github-actions
Copy link
Copy Markdown

빌드 결과

빌드 성공 🎉

@github-actions
Copy link
Copy Markdown

🎨 스토리북 배포 완료!

👉 스토리북 미리보기

변경된 컴포넌트의 디자인을 확인해주세요.

Copy link
Copy Markdown
Collaborator

@jisooooooooooo jisooooooooooo left a comment

Choose a reason for hiding this comment

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

수정하신 부분 확인했습니당!
전부 잘 작동하네요!!!👍🏻🔥
고생하셨어요오😙

Copy link
Copy Markdown
Contributor

@kwonsaebom kwonsaebom left a comment

Choose a reason for hiding this comment

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

확인했습니다 수고많으셨어요 😸 👍🏻👍🏻

@shinjigu shinjigu merged commit 940fcb3 into develop Sep 25, 2025
6 checks passed
@shinjigu shinjigu deleted the fix/#211/lowerTodo branch September 25, 2025 10:44
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.

[Fix]: QA 반영 - 하위 목표 뷰

3 participants