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

[team7 - FE] 이슈트래커 UI, 이슈 선택 동작, 이슈 생성 페이지 #38

Open
wants to merge 12 commits into
base: team7
Choose a base branch
from

Conversation

adelakim5
Copy link

진행사항

  • 이슈트래커 메인페이지, 생성페이지 UI
  • 체크 박스 선택 시 이슈 연결하기
  • 이슈생성 페이지에 마크다운 에디터 연결

앞으로 진행할 사항

  • 각 이슈별로 담당자, 레이블, 마일스톤, 작성자 filtering
  • 이슈 작성시 글자수 세기
  • API 요청, 응답 처리
  • 이슈 디테일 페이지
  • 테스트코드 작성..
  • 로그인, oauth..

느낀점

  • 최근 작업속도가 많이 더뎠습니다. 그래서 풀리퀘도 늦게 올리네요 ㅜㅜ 죄송합니다.
  • 이번에 코드를 짜면서 컴포넌트가 무거워지거나 제가 보기에 불편한 것들은 presenter-container 패턴으로 적용해서 구현해보았습니다.
  • presenter는 사실상 rendering만 담당한다고 알고있는데, 리코일을 사용하다보니 container에서 굳이 상태값을 프롭스로 내려주지 않아도 presenter에서 접근할 수 있고, 또 그게 뭔가 편하게 느껴지니까 기존의 presenter-container 패턴을 제가 깨부순것 같은 느낌도 듭니다.. 뭐가 맞는지 잘 모르겠네요 ㅜㅜ
  • 담당자,작성자,레이블, 마일스톤을 필터링 하는 부분이 개인적으로 어렵게 느껴지고 있습니다. 타입스크립트로 해서 그런지 object의 key값에 접근할 때 타입에러가 종종 나네요..
  • 시도중인 로직들은 주석으로 처리하였습니다. 다음에 마저 하려고 주석처리 하긴 했는데, 생각해보니 리뷰 하실 때 보기 불편할 수도 있을 것 같네요.. 죄송합니다 ㅜㅜ

구조

image

  • Home: 메인 페이지 (이슈 목록 조회되는 페이지)
  • AppBar: 이슈 목록 관리하는 메인 상단
  • IssueRef: 담당자, 마일스톤, 레이블, 작성자를 필터링하는 부분
  • IssueTable: 진짜 이슈 목록
  • Content: 에디터에서 제목, 내용을 작성하는 부분
  • Accordion: 담당자, 레이블, 마일스톤 등을 정하는 부분

리뷰어님 감사합니다!

머지할 때 스쿼시머지가 아닌 일반 머지로 부탁드릴게요! 🙏

adelakim5 and others added 9 commits June 14, 2021 23:38
- material-ui를 활용하여 뼈대 구성
- 아직 상태관리는 하나도 안함
    - 필터 관련해서 상태관리 설계 필요
- 리코일 공부해서 도입할 예정
- CheckBox 만들고 상태관리로 연동하는 시도중
- 목록마다 상태를 만들어줘야하나 고민
- 그러지 않아도 될 것 같은데.. 좀만 쉬다가 다시 해봐야지
- 체크박스 로직 구현
    - 전체 선택 시 모든 카드 선택되게 함
    - 부분 선택 취소 시 전체 선택 취소
    - 전체 선택 취소 시 모든 카드 취소
- 담당자, 레이블, 마일스톤, 작성자 버튼 수정
- 이슈 테이블 레이아웃 추가
- Chip 컴포넌트 쓰다가 color 커스터마이징이 안되어서 styled-component로 다시 구현중
- Editor 컴포넌트 스켈레톤 작업중
- 이슈작성 버튼 클릭 시 Editor 컴포넌트 나오도록 Link 연결
- 체크박스 체크 시 이슈 선택하는 상태 수정
- 체크박스 체크 시 AppBar 문구 변경
- ref filter 하는 부분 시도 중
@adelakim5 adelakim5 changed the title [team7] 이슈트래커 UI, 이슈 선택 동작, 이슈 생성 페이지 [team7 - FE] 이슈트래커 UI, 이슈 선택 동작, 이슈 생성 페이지 Jun 21, 2021
pbg0205 pushed a commit that referenced this pull request Jun 22, 2021
- 아코디언 부분 chip 제대로 수정
- 목록에서 아이템 클릭하면 클릭한 요소가 아코디언 타이틀에 뜨도록 구현
- 백엔드랑 라벨 부분 협의
- 오어스 없이 요청,응답 하는 식으로 임시 수정될 예정
- 그때까지만 휴식할 예정..
- api 수정중이라 잠시 중단
Sonjh1306 pushed a commit that referenced this pull request Jun 22, 2021
Security 사용하여 get, set 구현
테스트 코드 작성
crongro pushed a commit that referenced this pull request Jun 24, 2021
[FE] 이슈리스트 및 헤더 관련 추가
crongro pushed a commit that referenced this pull request Jun 25, 2021
crongro pushed a commit that referenced this pull request Jun 25, 2021
* 아이디, 비밀번호 input의 value.length가 0보다 클 때 로그인 버튼 활성화
* input type="password"
* git OAuth 로그인 fetch 요청, 토큰 받아서 로컬스토리지에 저장
crongro pushed a commit that referenced this pull request Jun 25, 2021
crongro pushed a commit that referenced this pull request Jun 25, 2021
* 로컬스토리지가 아닌 전역 상태에 정보를 저장하기 위해 새로고침을 하지 않는 방식으로 변경
* window.location.href 방식에서 useHistory를 사용하는 history.push로 변경
crongro pushed a commit that referenced this pull request Jun 25, 2021
crongro pushed a commit that referenced this pull request Jun 25, 2021
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.

1 participant