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

feature(admin): survey를 위한 viewmodel 작업 #2

Open
wants to merge 2 commits into
base: feat/admin
Choose a base branch
from

Conversation

Ssoon-m
Copy link
Collaborator

@Ssoon-m Ssoon-m commented Mar 1, 2024

작업한 내용

  • 옵션추가
  • 섹션추가
  • 설문제출
  • vanilla-extract, immer 라이브러리 추가

@Ssoon-m Ssoon-m changed the base branch from main to feat/admin March 1, 2024 01:30
@@ -0,0 +1,41 @@
import { vars } from '@ssoon-servey/shared-ui';

Choose a reason for hiding this comment

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

오 이거 직접 만드신건가요? 좋네요

Choose a reason for hiding this comment

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

p3: 여기 hook으로 빼서 사용하시는 것 좋은 것 같아요
validation 처리도 있으면 좋을 것 같은데요, 지금 훅에서는 validation을 통한 에러처리는 결합되지 않은 것 같아서 이부분도 같이 챙겨볼 수 있을까요?

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

Successfully merging this pull request may close these issues.

None yet

2 participants