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

sync: develop -> main #641

Merged
merged 7 commits into from
Nov 29, 2024
Merged

sync: develop -> main #641

merged 7 commits into from
Nov 29, 2024

Conversation

scarf005
Copy link
Member

No description provided.

YeonSeong-Lee and others added 7 commits October 3, 2024 20:35
* fix: 사용 불가능 카메라 목록에서 삭제 등

listVideoInputDevices()로 불러들인 디바이스 중 사용불가능한 것을 목록에서 제거
카메라 변경시마다 reset()을 호출

* Chore 인포메이션 집현전 위치 업데이트 (#613)

* ci: semantic PR 기준 완화

* WIP

* wip: Rending Confirm에서 사서가 자신의 책을 대출할 때 4권까지 대출 허용

lendingLimit 변수의 값이 selectedUser가 librarian(현재 로그인한 유저)인 경우 4로 설정됨
- prop으로 넘겨받고 있는데, Atom 에서 받아와서 설정하도록 다시 변경해야함

* fix: 책을 2권 초과하여 선택하는 경우 삭제 대상만 리스트에 남아있는 버그 수정

setSelectedBooks내부에서 불러오는 함수를  splice에서 filter로 변경

* wip: 임시로 4권 선택 가능하도록 수정해둔 상태

* fix: 사서인지 확인하는 변수를 prop이 아닌 userAtom으로 판단하도록 변경

* fix: 사서 본인의 대출인지 확인하는 방식을 prop에서 Atom으로 변경

* feat: 대출 유저 선택 후, 사서라면 사서 표시 추가

* feat: 사서 본인의 대출 시, 4권까지 가능하도록 lendingLimit 설정

* feat: 사서 본인의 대출에 따라 바뀌는 대출현황(/2권 -> /4권)

* refactor: 대출확인모달의 선택 도서 간 경계 및 비고란 명확한 표시 추가

* fix: penalty 표시에 괄호 안닫히던 bug 수정
Rent Modal User List와 Rent Inquire Box User 모두 연체인 경우만 괄호를 닫아주고 있어서 책 권수 초과에도 닫도록 조건 추가

* fix: 책 비고 작성시 2~4번째 책 비고란 묶여있던 거 각각 작성되도록 수정

remark(비고) array를 만들어서 selectedBooks의 index 이용하도록 설정

* fix: 이전 커밋 적용 된 뒤 "대출하기" 버튼 안눌리던 현상 수정

TextareaWithLabel에서 setState를 다시 생성해서 관리하고 있던 부분을 받아온 state를 이용하도록 변경

* fix: 유저의 role을 enum으로 변경 후 적용

* fix: Rent 비즈니스 로직 변경(대출권수)을 JSDoc으로 명시

* fix: lendingLimit 함수화 및 constant 적용

* fix: 변수 이름  수정, lending limit 함수 사용

피드백 반영해서
1. 변수 이름을 librarian에서 currUser로 수정했습니다.
2. lending limit의 경우 constant 폴더의 status에서 가져와서 사용하도록 했습니다.

* fix: lendingLimit 함수화 적용

* chore: using optional chaning

undefined 접근 에러를 예방하기 위해 옵셔널 체이닝을 추가합니다.

추후 필요에 따라 리팩토링과정에서 타입이 추가 될 수 있습니다.

* build: install react-helmet-async

for SEO

* feat: 기본적인 meta태그 설정

* feat: 책 상세페이지 meta tag 설정

* build: install puppeteer

for prerendeing, SEO

* chore: 오픈그래프 문구 수정

* build: 오픈그래프 테그를 위해 prerender 설정

* chore: update pnpm-lock.yaml

* ci: 자동배포용 스크립트

using codedeploy

* ci: install chrome for prerender

* fix: install chrome

---------

Co-authored-by: jhj9109 <jhj91_09@naver.com>
Co-authored-by: yena <50291995+nyj001012@users.noreply.github.com>
Co-authored-by: jhj9109 <36416495+jhj9109@users.noreply.github.com>
Co-authored-by: scarf <greenscarf005@gmail.com>
Co-authored-by: KIM CHANHEE <85754295+chanhihi@users.noreply.github.com>
Co-authored-by: hyeunkim laptop <hyeunkim@student.42seoul.kr>
Co-authored-by: Misung Kim <misung178@gmail.com>
ci: 자동배포 최종 main 머지
fix: `info/*` 로 직접접근하면 깨지는 버그 핫픽스
hotfix: `info/:id`에서 새로고침시 화면이 깨지는 오류 수정
build: 자잘한 버그 수정, 리렌더링 최적화 배포
#603 에서 추가한 이후로 info/{id} 경로 서빙되지 않음
@YeonSeong-Lee YeonSeong-Lee merged commit a0fa7a1 into develop Nov 29, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants