Skip to content

feat(fe): implement qna create page#2973

Merged
Seohwann merged 16 commits intomainfrom
t1949-create-qna-detailpage
Aug 28, 2025
Merged

feat(fe): implement qna create page#2973
Seohwann merged 16 commits intomainfrom
t1949-create-qna-detailpage

Conversation

@Seohwann
Copy link
Copy Markdown
Contributor

@Seohwann Seohwann commented Aug 13, 2025

Description

image figma 형식대로 create qna 페이지 구현완료했습니다.

Additional context

머리말

image

머리말은 General 텍스트 pixel을 기준으로,

image

만약 그보다 더 길다면 ... 으로 표현해두었으나 표현이 부자연스럽다면, 수정하겠습니다 !
❓ 머리말 너비가 변하면 UI가 커졌다가 작아졌다가 하기에 사용자가 느끼기에 부적절하다 생각해서 이렇게 구현했습니다.

POST 버튼 모달

image

현재 민규님께서 Modal을 만들어두셨는데 리뷰중이라 머지되면 해당 모달로 수정해놓겠습니다 !
현재는 위와 같은 기존의 모달을 임시적으로 사용하여 구현했습니다.

최대 텍스트 길이

image 유자차 Q&A 기능 명세서에 기재되어 있듯 제목은 70자, 질문글은 400자로 최대 글자수를 지정했습니다.

closes TAS-1949

Before submitting the PR, please make sure you do the following

@Seohwann Seohwann self-assigned this Aug 13, 2025
@github-project-automation github-project-automation Bot moved this to Pending ✋ in Codedang Aug 13, 2025
@notion-workspace
Copy link
Copy Markdown

@coolify-skkuding
Copy link
Copy Markdown

coolify-skkuding Bot commented Aug 13, 2025

The preview deployment failed. 🔴

Open Build Logs

Last updated at: 2025-08-27 07:31:01 CET

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 13, 2025

Syncing Preview App Failed

Application: frontend
Revision: a03b25198041c699f143d755ef0f0780b447c7b1
Health Status: Degraded

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 14, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: 86044db62fc16419c08f6ed8f88ef31a97366913
Health Status: Healthy

Open Preview | View in Argo CD

Copy link
Copy Markdown
Contributor

@Clover229 Clover229 left a comment

Choose a reason for hiding this comment

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

api 연결은 다른 pr에서 하셔도 되고 여기서 해도 될 것 같아요 ~ 디자인 굿입니다!

Comment thread apps/frontend/app/(client)/(main)/contest/[contestId]/@tabs/qna/page.tsx Outdated
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 17, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: 86044db62fc16419c08f6ed8f88ef31a97366913
Health Status: Healthy

Open Preview | View in Argo CD

@Seohwann
Copy link
Copy Markdown
Contributor Author

말씀해주신 부분들 일일이 확인해보고 있습니다 ! 수정하면 다시 리뷰 요청 보내드릴게요 !

@Seohwann
Copy link
Copy Markdown
Contributor Author

피드백해주신 부분들 모두 확인했습니다 !
다음 내용들 수정했습니다.

  1. API 연결 완료했습니다.
  2. 새로운 URL 페이지로 만들었습니다.
  3. useform,valibot 활용해서 다시 구현했습니다.
  4. 대회 시작전이라면 General만, 시작후라면 모든 문제 리스트를 나타내었습니다.
  5. 제목, 콘텐츠를 둘다 입력해야만 Post버튼이 활성화 되도록 했습니다.
  6. cancel버튼 -> 민규님께서 만드신 모달이 아직 머지 안돼서 머지되면 변경하겠습니다.
  7. maxlength 35로 수정했습니다.

Comment thread apps/frontend/app/(client)/(main)/contest/[contestId]/@tabs/qna/create/page.tsx Outdated
Comment thread apps/frontend/app/(client)/(main)/contest/[contestId]/@tabs/qna/create/page.tsx Outdated
@Seohwann
Copy link
Copy Markdown
Contributor Author

마지막으로 리팩토링까지 완료했습니다 ! 다시 한번씩만 리뷰해주시면 감사드리겠습니다 :)

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 26, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: e2946a561de6077290be294512e4f5013fb64a9b
Health Status: Healthy

Open Preview | View in Argo CD

@Seohwann Seohwann force-pushed the t1949-create-qna-detailpage branch from e2946a5 to 30b440a Compare August 26, 2025 07:04
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 26, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: 30b440abda5fb2ef3bc9db39d7607e4b5a66e7dd
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 26, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: 287fa437accb3c2882bd087e63d5789f5d5d5506
Health Status: Healthy

Open Preview | View in Argo CD

Copy link
Copy Markdown
Contributor

@Clover229 Clover229 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~짱!

@github-project-automation github-project-automation Bot moved this from Pending ✋ to Approved 👌 in Codedang Aug 27, 2025
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 27, 2025

Syncing Preview App Failed

Application: frontend
Revision: bc431b46742cbe954d1cdc9abc4fb4601261bd61
Health Status: Degraded

Open Preview | View in Argo CD

@sONg20NOW sONg20NOW self-requested a review August 27, 2025 16:24
Copy link
Copy Markdown
Contributor

@sONg20NOW sONg20NOW left a comment

Choose a reason for hiding this comment

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

머지를 축하합니다~ 🍰 LGTM

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 28, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: f1b02505dc6ec63b1faa40778b8815edc7478625
Health Status: Healthy

Open Preview | View in Argo CD

@Seohwann Seohwann force-pushed the t1949-create-qna-detailpage branch from f1b0250 to d0a7906 Compare August 28, 2025 02:16
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 28, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: d0a79065f3cf0554fe2c1feb98bb87c7bada8afe
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented Aug 28, 2025

Syncing Preview App Succeeded

Application: frontend
Revision: b8420f82d548c3e2f1970d5e0cd71473d6edac96
Health Status: Healthy

Open Preview | View in Argo CD

@Seohwann Seohwann added this pull request to the merge queue Aug 28, 2025
Merged via the queue into main with commit 46f3115 Aug 28, 2025
10 checks passed
@Seohwann Seohwann deleted the t1949-create-qna-detailpage branch August 28, 2025 07:26
@github-project-automation github-project-automation Bot moved this from Approved 👌 to Done ✔️ in Codedang Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done ✔️

Development

Successfully merging this pull request may close these issues.

3 participants