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

feat(lending): 사서의 대출 제한 권수 4권으로 조정 #826

Merged
merged 2 commits into from
Jul 12, 2024

Conversation

nyj001012
Copy link
Contributor

@nyj001012 nyj001012 commented Jul 12, 2024

개요

사서의 대출 제한 권수 4권으로 조정

작업 사항

lending.services.create에서 LENDING_OVERLOAD 반환 조건을 수정

변경점

사서는 4권, 일반 사용자는 2권까지 대출 가능

목적

스크린샷 (optional)

@nyj001012 nyj001012 added the enhancement New feature or request label Jul 12, 2024
@nyj001012 nyj001012 self-assigned this Jul 12, 2024
@nyj001012 nyj001012 linked an issue Jul 12, 2024 that may be closed by this pull request
1 task
Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

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

whitespace 수정이 많이 있는데, 혹시 prettier 적용되어있는지 확인해주실수 있나요?

@nyj001012
Copy link
Contributor Author

whitespace 수정이 많이 있는데, 혹시 prettier 적용되어있는지 확인해주실수 있나요?

prettier 적용해서 코드 정리 단축키 쓰니까 저렇게 수정됐네요

@scarf005
Copy link
Member

scarf005 commented Jul 12, 2024

whitespace 수정이 많이 있는데, 혹시 prettier 적용되어있는지 확인해주실수 있나요?

prettier 적용해서 코드 정리 단축키 쓰니까 저렇게 수정됐네요

image

image

혹시 prettier로 포매팅되었는지 다시 한번만 확인해 주실 수 있을까요? prettier playground의 실행 결과와 다르게 나오는 것 같습니다.

@nyj001012
Copy link
Contributor Author

whitespace 수정이 많이 있는데, 혹시 prettier 적용되어있는지 확인해주실수 있나요?

prettier 적용해서 코드 정리 단축키 쓰니까 저렇게 수정됐네요

image

image 혹시 prettier로 포매팅되었는지 다시 한번만 확인해 주실 수 있을까요? prettier playground의 실행 결과와 다르게 나오는 것 같습니다.

최신 커밋에 prettier 설정 바꿔서 재적용했는데, 괄호 띄어쓰기 적용 잘 된 것 같아요

@nyj001012 nyj001012 merged commit 255aaee into develop Jul 12, 2024
5 checks passed
@nyj001012 nyj001012 deleted the 825-사서들의-대출-제한-권수-추가 branch July 12, 2024 06:52
Copy link
Member

@YeonSeong-Lee YeonSeong-Lee left a comment

Choose a reason for hiding this comment

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

LGTM.

수고하셨습니다.

nyj001012 added a commit that referenced this pull request Jul 13, 2024
* feat(lending): 사서의 대출 제한 권수 4권으로 조정

* style(lending): prettier 재적용
nyj001012 added a commit that referenced this pull request Jul 13, 2024
* feat(lending): 사서의 대출 제한 권수 4권으로 조정

* style(lending): prettier 재적용
nyj001012 added a commit that referenced this pull request Jul 13, 2024
* feat(lending): 사서의 대출 제한 권수 4권으로 조정

* style(lending): prettier 재적용
nyj001012 added a commit that referenced this pull request Aug 18, 2024
* feat: 유저 search 할 때 id 가 undefined  인 경우 핸들링

* fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경

searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록

* [fix] backend dockerfile error (#764)

Co-authored-by: kylee <[email protected]>

* User API 경로 정리 (#777)

* refactor: 400번대 에러 반환 제거

* refactor: overDueDay 반환 값에서 제거

* fix: `dev/v2` 경로 복구 (#808)

* security: 보안 취약점 해결 (#818)

* feat(utils): rate limit 모듈 추가

- R, CUD에 해당하는 rate limit 모듈 추가

* refactor(cursus): rate limit 모듈 import해서 사용하도록 변경

* feat: getRateLimiter 적용

* feat(books): 유효한 ISBN인지 검사하는 로직 추가

* feat(tags): tags router에 rate limit 추가

* feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가

* feat(auth): /get/me에 rate limiter 추가

* feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용

* build: csrf 방지를 위한 lusca 패키지 추가

* feat(app): csrf 방지 로직 추가

* feat(app): csrf 방지 옵션 수정

* build: express-session 패키지 추가

* feat(app): lusca 상태 유지를 위한 세션 추가

* feat(app): cookie에도 secret 추가

* feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거

* User API 경로 정리 (#777)

* refactor: 400번대 에러 반환 제거

* refactor: overDueDay 반환 값에서 제거

* Revert "User API 경로 정리 (#777)"

This reverts commit 9adc701.

* feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826)

* feat(lending): 사서의 대출 제한 권수 4권으로 조정

* style(lending): prettier 재적용

---------

Co-authored-by: jimin <[email protected]>
Co-authored-by: gilee <[email protected]>
Co-authored-by: kylee <[email protected]>
Co-authored-by: scarf <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

사서들의 대출 제한 권수 추가
3 participants