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

도서 수정항목 추가 #516

Open
not-using opened this issue Jul 19, 2023 · 1 comment
Open

도서 수정항목 추가 #516

not-using opened this issue Jul 19, 2023 · 1 comment
Assignees
@mink97

Comments

@not-using
Copy link
Collaborator

기능 설명

기부자 정보 도 수정할 수 있도록
mink97 added a commit that referenced this issue Oct 30, 2023
@mink97
feat: 도서관리 - 기부자 수정 항목 부분 추가(#516-도서-수정항목-추가)
a64ceb2 
백엔드에 book/donator로 기부자 수정 url이 있어서 해당 url로 테스트.
-> 백엔드에서도 수정해야 할 부분 있음.
도서 수정시 요청을 두 개로 나누어서 보내야 하고, 추후 변경예정인 부분이 있어 보류
@mink97
Copy link
Contributor

mink97 commented Oct 30, 2023 •
edited
Loading

현재 백엔드에 있는 book/donator url을 통해 요청하여 테스트.

  • 원래 도서 관리 탭에서 book/update를 통해 한 번에 수정사항을 업데이트 하였는데 기부자 업데이트 방식은 현재 분리되어있는 상태

기부자 변경 로직

  • 기부자 닉네임을 등록된 유저 목록에서 찾아보고 해당하는 유저가 없으면 오류 리턴
    • 외부에서 기증할 수도 있기 때문에 유저 목록에 없어도 등록 가능하도록 변경 예정이라고 함.
  • 변경하는 닉네임 문자열을 포함하는 유저들 중에서 제일 앞에 있는 유저를 가져오는 듯? -> 예상치 못한 결과 나올 수 있음

때문에 후에 v2에서 구현완료되면 그 때 다시 작업하도록 하겠습니다,

@mink97 mink97 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 30, 2023
@mink97 mink97 reopened this Oct 30, 2023
@mink97 mink97 pinned this issue Oct 30, 2023
@mink97 mink97 unpinned this issue Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mink97 mink97

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

516-도서-수정항목-추가 jiphyeonjeon-42/frontend
2 participants
@not-using @mink97