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

남의위키 설문작성 1-4번 OX 선지 항목 구현 #40

Merged
merged 11 commits into from
Feb 20, 2024

Conversation

spolice0324
Copy link
Member

Issue No.

#36


작업 내역

구현 사항 및 작업 내역

  • 1 OX 2지 선다 문항 1-4번 항목 구현
  • 2 inputLabel 수정 - required 추가, 타이포 변동 대비 className props 변경
  • 3 api 설문 조회 연동 전 목데이터로 option 작업
  • 4 이유를 말해주세요 Input 추가

세부사항

서버 통신이 어려운 상태인 거 같아 목데이터로 해당 설문 문항 작업 구현했습니다.
프로필 조회 api가 나오지 않은 상태라 userName과 title 또한 dangerouslySetInnerHTML로 감싸, 보이는 형태만 얼추 꾸려놓은 상태입니다. 해당 부분은 api 연동후 닉네임을 치환하는 걸로 결정된 부분입니다.
progress bar와 SHORT_ANSWER 선지들을 추가하면서 응답에 맞는 interface 와 submit Value들을 수정할 예정입니다.

화면 결과 (캡쳐 첨부)

2024-02-20.3.57.59.mov

@spolice0324 spolice0324 added this to the 설문 폼 구현 milestone Feb 20, 2024
@spolice0324 spolice0324 self-assigned this Feb 20, 2024
Copy link

vercel bot commented Feb 20, 2024

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

Name Status Preview Comments Updated (UTC)
dnd-10th-6-frontend ❌ Failed (Inspect) Feb 20, 2024 7:28am

@spolice0324 spolice0324 linked an issue Feb 20, 2024 that may be closed by this pull request
5 tasks
@spolice0324 spolice0324 removed a link to an issue Feb 20, 2024
5 tasks
Comment on lines +44 to +48
<div
dangerouslySetInnerHTML={{
__html: data.title,
}}
></div>
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

오,,, api 나오면 해당 방법 활용해서 구현해보겠슴니다 !!

@spolice0324 spolice0324 merged commit 74d47ec into main Feb 20, 2024
1 of 3 checks passed
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.

2 participants