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: Diary content,font 삭제 #24

Merged
merged 5 commits into from
Nov 25, 2023
Merged

fix: Diary content,font 삭제 #24

merged 5 commits into from
Nov 25, 2023

Conversation

0703kyj
Copy link
Contributor

@0703kyj 0703kyj commented Nov 23, 2023

개요

  • Diary에서 image 제외한 부분 삭제했습니다.

작업사항

  • Diary 관련 클래스 content,font 부분 삭제

주의사항

  • api 테스트도 문제없이 작동하는 것 확인했습니다.
  • 빼먹은 부분이 있다면 알려주세요!

@0703kyj 0703kyj self-assigned this Nov 23, 2023
@0703kyj 0703kyj added the ✨ 기능 구현 기능 구현 label Nov 23, 2023
@0703kyj 0703kyj requested a review from Ji-soo708 November 23, 2023 14:11
Copy link
Member

@Ji-soo708 Ji-soo708 left a comment

Choose a reason for hiding this comment

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

수정사항 확인했습니다! 그리고 일기 수정 기능도 이제 안쓰여서 해당 API 삭제해도 좋을 거 같아요~

@0703kyj 0703kyj added the 🚨버그🚨 버그 해결 label Nov 23, 2023
@0703kyj
Copy link
Contributor Author

0703kyj commented Nov 23, 2023

이미지 자체를 변경하는 식으로 사용할수도 있을 것 같아서 혹시 몰라 남겨두는게 좋을 것 같아서 삭제 안했었습니다! 일단은 놔두는게 좋을 것 같은데 어떻게 생각하세요?

@Ji-soo708
Copy link
Member

이미지 자체를 변경하는 식으로 사용할수도 있을 것 같아서 혹시 몰라 남겨두는게 좋을 것 같아서 삭제 안했었습니다! 일단은 놔두는게 좋을 것 같은데 어떻게 생각하세요?

아 그러면 일단 두고 프론트분들과 한 번 더 얘기하고 정하는 게 좋겠네요 👍🏻

@Ji-soo708 Ji-soo708 added ✏️ API 변경 API 변경 and removed ✨ 기능 구현 기능 구현 labels Nov 24, 2023
@0703kyj 0703kyj merged commit 87a5e12 into main Nov 25, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✏️ API 변경 API 변경 🚨버그🚨 버그 해결
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants