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

[ISSUE-104/bugfix] 가상계좌 관련 API 호출 경로 수정 #105

Merged
merged 3 commits into from
Oct 16, 2023
Merged

[ISSUE-104/bugfix] 가상계좌 관련 API 호출 경로 수정 #105

merged 3 commits into from
Oct 16, 2023

Conversation

ImKunYoung
Copy link
Contributor

@ImKunYoung ImKunYoung commented Oct 16, 2023 •
edited
Loading

  Client 수정된 Client Admin Product 수정된 Product
가상계좌 추가 /admin/product/vbanks /admin/vbanks /product/vbanks /admin/vbanks  
가상계좌 삭제 /admin/product/vbanks/{vBankId} /admin/vbanks/{vBankId} /product/vbanks/{vBankId} /admin/vbanks/{vBankId}  
가상계좌 수정 /admin/product/vbanks/{vBankId} /admin/vbanks/{vBankId} /product/vbanks/{vBankId} /admin/vbanks/{vBankId}  
가상계좌 리스트 조회 /admin/product/vbanks /product/vbanks /product/vbanks /vbanks  


스크린샷

수정 전
image

image



수정 후

가상계좌 리스트 조회
image


가상계좌 추가
image


가상계좌 수정
image


가상계좌 삭제
image

Closes #104
Closes #106

@ImKunYoung ImKunYoung added the 🐞 BugFix Something isn't working label Oct 16, 2023
@ImKunYoung ImKunYoung self-assigned this Oct 16, 2023
@ImKunYoung ImKunYoung linked an issue Oct 16, 2023 that may be closed by this pull request
[Bug] 가상계좌 관련 API 호출 경로 수정 #104
Closed
@ImKunYoung ImKunYoung changed the title bugfix(#104): Modified API Paths for Virtual Account Management. [ISSUE-104/bugfix] 가상계좌 관련 API 호출 경로 수정 Oct 16, 2023
DryRains
DryRains approved these changes Oct 16, 2023
View reviewed changes
Copy link
Member

@DryRains DryRains left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !!

@ImKunYoung ImKunYoung merged commit b0a0129 into main Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DryRains DryRains DryRains approved these changes

@ansm6358 ansm6358 Awaiting requested review from ansm6358

@heeoneie heeoneie Awaiting requested review from heeoneie

@2hanbyeol1 2hanbyeol1 Awaiting requested review from 2hanbyeol1

@jungyeons jungyeons Awaiting requested review from jungyeons

@fogus14 fogus14 Awaiting requested review from fogus14

@KimHyeongKeun KimHyeongKeun Awaiting requested review from KimHyeongKeun

Assignees

@ImKunYoung ImKunYoung

Labels
🐞 BugFix Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug] 가상계좌 주문 상태 수정 400 에러 [Bug] 가상계좌 관련 API 호출 경로 수정
2 participants
@ImKunYoung @DryRains