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.

Sign up for GitHub

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

Jump to bottom

refactor: V1 API 제거 #332

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from
Open

refactor: V1 API 제거 #332

wants to merge 16 commits into from

Conversation

huGgW
Copy link
Collaborator

@huGgW huGgW commented Jan 14, 2025

변경점

  • deprecated된 api 제거, 기존 v1 api 모두 v2 api로 이동

추가사항

  • file 관련 api는 일단 그대로 유지하였습니다.
    • db상에 url 그대로 사용되고 있어 수정시 db 마이그레이션 필요
    • 수정할지 그대로 유지할지 결정 필요

@huGgW huGgW requested a review from leeeryboy January 14, 2025 15:29
@huGgW huGgW self-assigned this Jan 14, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 14, 2025 •
edited
Loading

Test Results

11 files  11 suites   1s ⏱️
54 tests 54 ✔️ 0 💤 0 ❌
58 runs  58 ✔️ 0 💤 0 ❌

Results for commit c1ee37d.

♻️ This comment has been updated with latest results.

leeeryboy
leeeryboy approved these changes Jan 19, 2025
View reviewed changes
Copy link
Contributor

@leeeryboy leeeryboy left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leeeryboy leeeryboy leeeryboy approved these changes

Assignees

@huGgW huGgW

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@huGgW @leeeryboy