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

fix(fe): hand over only changed field in problem edit #2098

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

B0XERCAT
Copy link
Member

Description

기존에 Problem Edit에서는 필드 수정 여부와 상관 없이 모든 필드를 전달했으나
수정된 필드만 전달되도록 수정햇습니다

Additional context

tags, testcases, template의 경우 ProblemWithIsVisible과 UpdateProblemInput 스키마에 대해 논의가 필요할 것 같습니다


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

@B0XERCAT B0XERCAT added ⛳️ team-frontend 💡 feature-admin Features for admin & group manager labels Sep 19, 2024
@B0XERCAT B0XERCAT self-assigned this Sep 19, 2024
Copy link

vercel bot commented Sep 19, 2024

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

Name Status Preview Updated (UTC)
codedang ❌ Failed (Inspect) Sep 19, 2024 6:03am

@Jaehyeon1020
Copy link
Member

백엔드도 작업 후 머지할게요!

@B0XERCAT
Copy link
Member Author

백엔드도 작업 후 머지할게요!

넵 알겠습니다!

@jimin9038
Copy link
Member

coolify 배포를 위해서 닫았다가 열게요.

@jimin9038 jimin9038 closed this Sep 26, 2024
@jimin9038 jimin9038 reopened this Sep 26, 2024
@coolify-skkuding
Copy link

coolify-skkuding bot commented Sep 26, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-09-26 09:31:10 CET

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 feature-admin Features for admin & group manager ⛳️ team-frontend
Projects
Status: Pending
Development

Successfully merging this pull request may close these issues.

4 participants