[7주차] 셰어마인드 미션 제출합니다. #7
Open
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
- package.json, tsconfig.json 설정, 폰트 설정, custom.d.ts module 선언
build: eslintrc delete cr 해결
[feat] 공통컴포넌트 추가, 기초세팅, 회원가입 스타일링 완료
[feature] 회원가입 구현완료
전체적인 디자인 수정 및 투표결과 UI view 구현
[feature] 투표 후보 선택까지 완료
Fix: 투표 선택 오류 해결
Fix: 투표 선택 오류 해결2
wokbjso
reviewed
Dec 28, 2023
Comment on lines
+6
to
+48
|
|
||
| export const getInstance = async (url: string, params?: any, header?: any) => { | ||
| try { | ||
| const data = await instance.get(url, params); | ||
| return data; | ||
| } catch (error) { | ||
| return error; | ||
| } | ||
| }; | ||
| export const postInstance = async (url: string, body: any, params?: any) => { | ||
| try { | ||
| const data = await instance.post(url, body, params); | ||
| return data; | ||
| } catch (error) { | ||
| return error; | ||
| } | ||
| }; | ||
| export const putInstance = async (url: string, body: any, params: any) => { | ||
| try { | ||
| const data = await instance.put(url, body, params); | ||
| return data; | ||
| } catch (error) { | ||
| return error; | ||
| } | ||
| }; | ||
|
|
||
| export const patchInstance = async (url: string, body: any, params: any) => { | ||
| try { | ||
| const data = await instance.patch(url, body, params); | ||
| return data; | ||
| } catch (error) { | ||
| return error; | ||
| } | ||
| }; | ||
|
|
||
| export const deleteInstance = async (url: string) => { | ||
| try { | ||
| const data = await instance.delete(url); | ||
| return data; | ||
| } catch (error) { | ||
| return error; | ||
| } | ||
| }; |
There was a problem hiding this comment.
메소드를 제외한 코드가 중복되므로 외부에서 method 도 매개변수로 받아온 뒤 try 문에서 실행되는 메소드를 method의 값에 따라 분기처리해도 좋을 것 같아요 ㅎㅎ
src/assets/images/FE.png
Outdated
Comment on lines
+18
to
+19
| <VoteItem | ||
| key={index} |
There was a problem hiding this comment.
key를 index 단독으로 지정하는 것을 추천하지 않는다고 하는데 한번 알아보시면 좋을 것 같아용 ㅎㅎ
| @@ -0,0 +1,73 @@ | |||
| import { useState } from 'react'; | |||
Comment on lines
+27
to
+31
| const [cookies, setCookie, removeCookie] = useCookies([ | ||
| 'userEmail', | ||
| 'userPassword', | ||
| ]); | ||
|
|
There was a problem hiding this comment.
쿠키 사용도 시도하셨네요!! 저희는 시간이 없어 도입을 못했지만 프로젝트에서 도입해보겠습니다 ㅎㅎ
| @@ -0,0 +1,28 @@ | |||
| @font-face { | |||
oooppq
reviewed
Dec 28, 2023
| import styled from 'styled-components'; | ||
| import { PartCandidateArrayType } from 'utils/type'; | ||
|
|
||
| function VoteResultBack() { |
There was a problem hiding this comment.
VoteResultFront와 거의 대부분 겹치는 컴포넌트인 것 같아요! 추상화해서 하나의 컴포넌트로 만들어도 좋을 것 같습니다~~
| ); | ||
| useEffect(() => { | ||
| const fetchCandidateDemo = async () => { | ||
| const res: any = await getDemoday(); |
There was a problem hiding this comment.
타입스크립트의 장점을 극대화하려면 any를 지양하고 최대한 타입 지정해주는 것이 좋을 것 같아요~~
Comment on lines
+19
to
+20
| const partSelect = useCustomSelect('', 'part'); | ||
| const teamSelect = useCustomSelect('', 'team'); |
| return error; | ||
| } | ||
| }; | ||
| export const postInstance = async (url: string, body: any, params?: any) => { |
There was a problem hiding this comment.
param이나 body 부분도 특정 형태의 데이터를 보내도록 어느정도 정해져있기 때문에 구체적으로 타입지정을 하게 된다면 api 통신함에 있어서의 에러를 방지할 수 있을 것 같습니다~~
Comment on lines
+20
to
+42
| <TopRankBarList> | ||
| <SecondBar> | ||
| <SecondWho> | ||
| <Name>{candidateBE[1]?.name}</Name> | ||
| <Team>{candidateBE[1]?.team}</Team> | ||
| </SecondWho> | ||
| <Number>2</Number> | ||
| </SecondBar> | ||
| <FirstBar> | ||
| <Number>1</Number> | ||
| <FirstWho> | ||
| <Name style={{ color: 'gold' }}>{candidateBE[0]?.name}</Name> | ||
| <Team>{candidateBE[0]?.team}</Team> | ||
| </FirstWho> | ||
| </FirstBar> | ||
| <ThirdBar> | ||
| <Number>3</Number> | ||
| <ThirdWho> | ||
| <Name>{candidateBE[2]?.name}</Name> | ||
| <Team>{candidateBE[2]?.team}</Team> | ||
| </ThirdWho> | ||
| </ThirdBar> | ||
| </TopRankBarList> |
flowerseok
reviewed
Dec 28, 2023
| // setSelectedCandIdFE(-1); | ||
| const result = window.confirm( | ||
| '투표를 행사할 수 있는 권리는 한 번이며, 번복이 불가능합니다. \n이대로 진행하시겠습니까?', | ||
| ); |
Fix: 데모데이 투표 겹치는 오류 해결
geeoneee
reviewed
Dec 29, 2023
| await postInstance('/email', body); | ||
| //로그인 아이디 중복 체크 | ||
| export const postCheckId = async (body: any) => | ||
| await postInstance('/user/loginId', body); |
There was a problem hiding this comment.
이렇게 정리하니까 postInstance 한 눈에 보기 좋은 것 같습니다! 배워가용
| alert('error'); | ||
| return; | ||
| } | ||
| console.log(value); |
| else setMessage('팀을 선택해주세요'); | ||
| } else { | ||
| setBgColor('#ffffff'); | ||
| setBdColor('#cccccc'); |
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.
배포주소
React-vote
디자인
CEOS 홈페이지 베끼기 ㅎㅎ..
피그마 링크
구현한 것
협업 과정
백엔드에서 Swagger 를 통하여 API 명세서를 잘 만들어주었고, 운이 좋게 명세서 안에서 작업이 잘 이루어졌던 거 같습니다.
느낀 점
전체적으로 책이 넘겨지는 것 같은 트랜지션을 주려고 한 페이지에 컴포넌트를 우겨넘은 감이 있는데, 시간적으로 여유가 없어서 구현을 못한 게 아쉬웠습니다.