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

[시간표] 삭제 되돌리기 기능 #510

Merged
merged 19 commits into from
Sep 26, 2024
Merged

Conversation

Gwak-Seungju
Copy link
Contributor

@Gwak-Seungju Gwak-Seungju commented Sep 24, 2024

What is this PR? 🔍

Changes 📝

  • api path 안 맞는 부분 수정했습니다.
  • 되돌리기 로직과 토스트 띄우는 로직을 삭제 hook의 onSuccess로 옮겼습니다.
  • 토스트가 띄워지는 위치를 아래에서 위로 수정했습니다.
  • 아래 파일이 되돌리기 기능 구현의 주 파일입니다. 이를 위주로 봐주시면 감사하겠습니다.
    src/pages/TimetablePage/hooks/useDeleteTimetableFrame.ts
    src/pages/TimetablePage/hooks/useTimetableV2Mutation.ts

  • 추가로 원래 학기를 삭제하는 경우에도 복구 기능을 추가하려 했지만, 삭제 후 삭제 확인 모달이 있어서 추가하지 않았습니다.
  • 또한, 현재는 강의 복구를 할 때 분반, 학과코드 등이 표시되지 않고 있고, 시간표 프레임도 복구할 때 이전 이름으로 복구되지 않는데 이는 api 수정 이후 잘 작동할 예정입니다.(api 수정 이후 한 번 더 확인하겠습니다.) --> 작동 잘 됩니다! 4ad645a

ScreenShot 📷

  • 강의 삭제 후 되돌리기 시
-.-.Chrome.2024-09-24.12-53-41.mp4
  • 시간표 프레임 삭제 후 되돌리기 시
-.-.Chrome.2024-09-24.12-56-00.mp4

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

@Gwak-Seungju Gwak-Seungju added ✨ Feature 기능 개발 👤 User 유저, 시간표 도메인 labels Sep 24, 2024
@Gwak-Seungju Gwak-Seungju self-assigned this Sep 24, 2024
Copy link
Contributor

@MinGu-Jeong MinGu-Jeong left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

@Gwak-Seungju Gwak-Seungju merged commit 0a75f00 into develop Sep 26, 2024
2 checks passed
@github-actions github-actions bot deleted the feat/#495/revert-item branch September 26, 2024 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 👤 User 유저, 시간표 도메인
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[시간표] 강의, 시간표 프레임, 학기 삭제 시 되돌리기 기능 구현
2 participants