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

학습자료 crud api 연결 #297

Merged
merged 30 commits into from
Sep 4, 2024
Merged

학습자료 crud api 연결 #297

merged 30 commits into from
Sep 4, 2024

Conversation

mun-kyeong
Copy link
Collaborator

관련 이슈

작업 요약

  • 학습자료 관련 상세 수정 완료했습니다

작업 상세 설명

  • 모달 전체의 placeholder 색상 white로 수정
  • 학습 자료 생성 모달의 font-weigth 동일하게 맞추기
  • 첨부가능한 파일의 종류 제한두기 (.cpp, .py 같은 파일은 백엔드 에서 막아두어 생성이 안 됨.)
  • 학습자료 url, 파일의 미리보기 아이콘 수정
  • 이미지와 링크 클릭 시 잘못된 url로 라우팅 됨.
  • 링크 학습자료에 링크 여러개 첨부 가능하지만 조회의 1개만 됨.
  • 학습자료 첨부된 링크 누르면 AccessDenied됨….. ⇒ 프론트에서 s3 링크를 같이 붙여서 보내줌
  • 학습자료 수정한다음에 그냥 확인 누르면 제목이랑 설명 다 날아감

리뷰 요구 사항

  • 수정 필요한 부분 있다면 코멘트 부탁드립니다!
  • 브랜치를 옮기지 않고 그대로 작업을 진행해버려서,,! 이전 pr 올렸던 브랜치라고 생각해주시면 돼요
    (늦게 알아버려서 ㅠㅠ 죄송합니다,,!)

미리 보기

image image

mun-kyeong and others added 29 commits June 29, 2024 23:31
@mun-kyeong
feat : 버튼 클릭시 모달 오픈 연결
1df53bd
@mun-kyeong
feat : 파일 받아오는거까지 확인
a3b437c
@mun-kyeong
Merge branch 'develop' of https://github.com/BDD-CLUB/01-doo-re-front …
183f18b 
…into feature/#253-학습자료-C-API-연결-
@mun-kyeong
feat : api 전송 확인
ac6d9fb
@mun-kyeong
Merge branch 'develop' of https://github.com/BDD-CLUB/01-doo-re-front …
ffe9028 
…into feature/#253-학습자료-C-API-연결-
@mun-kyeong
feat : 팀 페이지에서 카테고리 변경시 데이터 받아와지지 않는 문제 해결
6c7679b
@mun-kyeong
feat : 주석처리
6ed6019
@mun-kyeong
feat : img, url 전송 확인, 근데 file 전송 실패 이유는 잘 모르겠음...
4feb2e7
@mun-kyeong
feat : 학습자료카드 bookmark정보 삭제 / mocks와 DTO 형식 통일
5e72b74
@mun-kyeong
feat : getDocumentList 성공
840cdb6
@mun-kyeong
feat : 학습자료 post teamId 적용
d02802a
@mun-kyeong
feat : 팀페이지 + 버튼 누르면 모달창 오픈
de259a3 
- 팀 id 다를때에도 제대로 들어오는거 확인됨
@mun-kyeong
feat : 학습자료 상세 페이지 get 완료
2f7c960
@mun-kyeong
Merge branch 'develop' of https://github.com/BDD-CLUB/01-doo-re-front …
0dd5395 
…into feature/#253-학습자료-CRUD-API-연결-
@mun-kyeong
refactor : 주석정리
eb2f30e
@mun-kyeong
feat : 스터디 / 팀별 학습자료 post, get 확인
8875c1e
@mun-kyeong
feat : 미리보기 img 준비
c9df590
@mun-kyeong
feat : 팀 페이지 학습자료 페이지 생성 + delete api 전송 확인
1b3b0c2 
- delete 이후 페이지 update 필요
@mun-kyeong
api : getDocumentList api 수정 반영 완료 + page navigator 작동 확인
7547fc3
@mun-kyeong
feat : put Document api 정리
e3a892c
@mun-kyeong
Merge branch 'develop' of https://github.com/BDD-CLUB/01-doo-re-front …
08bdc27 
…into feature/#253-학습자료-CRUD-API-연결-
@mun-kyeong
feat : 수정 버튼 클릭시 수정 모달 open
488f10c
@mun-kyeong
feat : 학습자료 api 무한요청 => useEffect 수정
db139dd
@mun-kyeong
feat : console 정리
011b39c
@mun-kyeong
feat : put 수정 확인 : 화면 update 필요
4a9f7c5
@mun-kyeong
feat : 미리보기 이미지 적용
8078b03
@mun-kyeong
feat : mocks 수정
82794bf
@mun-kyeong
refactor : 미리보기 이미지 변경 및 디자인 수정
5d59823
@mun-kyeong
refactor : 수정시 이전 기록 가져오기
129ff17
@mun-kyeong mun-kyeong self-assigned this Aug 15, 2024
@mun-kyeong
Merge branch 'develop' of https://github.com/BDD-CLUB/01-doo-re-front …
c2563e3 
…into feature/#253-학습자료-CRUD-API-연결-

# Conflicts:
#	public/png/file.png
#	public/png/folder.png
#	src/components/DocumentCard/index.tsx
#	src/containers/study/CreateDocumentModal/index.tsx
#	src/containers/study/DocumentModal/index.tsx
llddang
llddang approved these changes Aug 22, 2024
View reviewed changes
Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

jasper200207
jasper200207 approved these changes Aug 22, 2024
View reviewed changes
Copy link
Collaborator

@jasper200207 jasper200207 left a comment

Choose a reason for hiding this comment

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

👍

@mun-kyeong mun-kyeong merged commit cedeaec into develop Sep 4, 2024
1 check passed
@mun-kyeong mun-kyeong deleted the feature/#253-학습자료-CRUD-API-연결- branch September 4, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@llddang llddang llddang approved these changes

@jasper200207 jasper200207 jasper200207 approved these changes

@pipisebastian pipisebastian Awaiting requested review from pipisebastian

@yeonddori yeonddori Awaiting requested review from yeonddori

Assignees

@mun-kyeong mun-kyeong

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 학습자료 CR API 연결
3 participants
@mun-kyeong @jasper200207 @llddang