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

[FE] #12 주식주문 관련 컴포넌트 레이아웃 및 기능 구현 #39

Merged
merged 12 commits into from
Sep 4, 2023
Merged

[FE] #12 주식주문 관련 컴포넌트 레이아웃 및 기능 구현 #39

merged 12 commits into from
Sep 4, 2023

Conversation

novice1993
Copy link
Contributor

@novice1993 novice1993 commented Sep 4, 2023 •
edited
Loading

  • 우측 주식주문 관련 컴포넌트 레이아웃 및 기능 일부 구현
  • 일부 코드 다듬은 후 나머지 부분 구현 예정

@novice1993
[Feat] 주식주문 컴포넌트 레이아웃 일부 구현
8203e04 
- 주식주문 컴포넌트 (매수/매도 버튼 클릭시 렌더링 되는 화면) 레이아웃 일부 구현
- 상단바 구현

Issues #12
@novice1993
[Feat] 주식주문 컴포넌트 상단 종목명 레이아웃 구현
679c2b7 
- 주식주문 컴포넌트 상단 종목명 관련 레이아웃 구현

Issues #12
@novice1993
[Feat] 주가 및 주문설정 관련 레이아웃 일부 구현
2e77a51 
- 주가 및 주문설정 관련 전체 레이아웃 설정
- 레이아웃 내부에 들어가는 요소 구현 중

Issues #12
@novice1993
[Feat] 주식주문 컴포넌트 전체 레이아웃 틀 구현
323b525 
- 주식주문 컴포넌트 전체 레이아웃 틀 구현 완료
- 세부적인 요소 및 디자인 추가 예정

Issues #12
@novice1993
[Feat] 브랜치 병합
36335b9 
- 로그인/로그아웃 관련 컴포넌트 구현으로 인한 브랜치 병합

Issues #12
@novice1993
[Design] 일부 컴포넌트 CSS 속성 변경
7c156d7 
- 가격 및 거래량 컨트롤러 관련 CSS 속성 변경

Issues #12
@novice1993
[Design] 일부 컴포넌트 CSS 수정
cd6f9a4 
- 가격/거래량 컨트롤러 관련 컴포넌트 CSS 수정

Issues #12
@novice1993
[Feat] 주가 정보관련 화면 구현
8f4d77c 
- 주식주문 컴포넌트의 주가 정보관련 화면 구현
- 세부 기능 추가 예정

Issues #12
@novice1993
[Design] 주가목록 관련 CSS 수정
6ddcc66 
- 렌더링 시, 중간가격 주가가 리스트 정중앙에 렌더링 되도록 Reack hook 및 css 관련 속성 부여

Issues #12
@novice1993
[Feat] 주식주문 관련 조건부 렌더링 적용
a42684b 
- 주식주문 관련 매수/매도 여부에 따른 조건부 렌더링 일부 구현
- 전역 상태관리도구 (redux-toolkit) 활용하여 상태관리

Issues #12
@novice1993
[Feat] 주가 선택 및 변경 기능 구현
7764e4d 
- 주가 선택 및 변경 기능 일부 구현
- 지정가/시장가 변환 관련 기능 구현 예정

Issues 12
@novice1993
[Feat] 브랜치 병합
7a700ac 
- 좌측 비교차트 관련 컴포넌트 구현이로 인한 브랜치 병합

Issues #12
@novice1993 novice1993 added the FE Front End label Sep 4, 2023
@novice1993 novice1993 added this to the Bare Minimum Requirement milestone Sep 4, 2023
@novice1993 novice1993 self-assigned this Sep 4, 2023
@novice1993 novice1993 changed the title [FE] #12 주식주문 관련 컴포넌트 레이아웃 구현 [FE] #12 주식주문 관련 컴포넌트 레이아웃 및 기능 구현 Sep 4, 2023
@sirloinbh
Copy link
Collaborator

네 코드 다 확인했습니다.

@sinjw
Copy link
Collaborator

sinjw commented Sep 4, 2023

네 확인했습니다 !

@paul01010
Copy link

@sinjw @sirloinbh 님~
지금 댓글로 작성해주시는 방법도 좋지만.. 코드리뷰가 완료되었으면, 여기 approve를 눌러주시는게 더 좋을것같아요~ ㅎㅎ
image

@sirloinbh
Copy link
Collaborator

approve 기능도 있었군요

@sinjw
Copy link
Collaborator

sinjw commented Sep 4, 2023

오오 네확인했습니다

@novice1993 novice1993 merged commit 377a3c7 into dev-client Sep 4, 2023
@novice1993 novice1993 deleted the dev-client#12/stockOrderLayout branch September 4, 2023 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sinjw sinjw sinjw approved these changes

@sirloinbh sirloinbh sirloinbh approved these changes

Assignees

@novice1993 novice1993

Labels
FE Front End
Projects
None yet
Milestone
Bare Minimum Requirement
Development

Successfully merging this pull request may close these issues.
4 participants
@novice1993 @sirloinbh @sinjw @paul01010