-
Notifications
You must be signed in to change notification settings - Fork 37
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
adelakim5
wants to merge
12
commits into
codesquad-members-2021:team7
Choose a base branch
from
bibi6666667:fe/feature/editor-skeleton
base: team7
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[team7 - FE] 이슈트래커 UI, 이슈 선택 동작, 이슈 생성 페이지 #38
adelakim5
wants to merge
12
commits into
codesquad-members-2021:team7
from
bibi6666667:fe/feature/editor-skeleton
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- material-ui를 활용하여 뼈대 구성 - 아직 상태관리는 하나도 안함 - 필터 관련해서 상태관리 설계 필요 - 리코일 공부해서 도입할 예정
- CheckBox 만들고 상태관리로 연동하는 시도중 - 목록마다 상태를 만들어줘야하나 고민 - 그러지 않아도 될 것 같은데.. 좀만 쉬다가 다시 해봐야지
- 체크박스 로직 구현 - 전체 선택 시 모든 카드 선택되게 함 - 부분 선택 취소 시 전체 선택 취소 - 전체 선택 취소 시 모든 카드 취소
- 담당자, 레이블, 마일스톤, 작성자 버튼 수정 - 이슈 테이블 레이아웃 추가
- Chip 컴포넌트 쓰다가 color 커스터마이징이 안되어서 styled-component로 다시 구현중
[아델라]Fe/feature/skeleton
- Editor 컴포넌트 스켈레톤 작업중 - 이슈작성 버튼 클릭 시 Editor 컴포넌트 나오도록 Link 연결
[아델라]feat: Editor-skeleton
- 체크박스 체크 시 이슈 선택하는 상태 수정 - 체크박스 체크 시 AppBar 문구 변경 - ref filter 하는 부분 시도 중
adelakim5
changed the title
[team7] 이슈트래커 UI, 이슈 선택 동작, 이슈 생성 페이지
[team7 - FE] 이슈트래커 UI, 이슈 선택 동작, 이슈 생성 페이지
Jun 21, 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 25, 2021
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
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
진행사항
앞으로 진행할 사항
느낀점
구조
리뷰어님 감사합니다!
머지할 때 스쿼시머지가 아닌 일반 머지로 부탁드릴게요! 🙏