SOTATER / back-end

3 stars 0 forks source link

GET Profile Data #39

Open minyoungyang opened 1 year ago

minyoungyang commented 1 year ago

GET /profile

Request Parameter (query) parameter type required description
searchText string true 소환사 이름
Response field type description
id number | null 해당 소환사가 없다면 null, 나머지 필드 없이
currentTier Tier 현재 티어 - 프로필 이미지의 border를 표현하기 위함
pastRank SummonerPastRank[] 과거 랭크 정보 - 없다면 빈 배열
profileId number 프로필 이미지 ID
level number 현재 레벨
ranking SummonerRanking 랭킹 정보
updated string 업데이트 된 시점
export type Tier =
    | 'BRONZE'
    | 'CHALLENGER'
    | 'DIAMOND'
    | 'GOLD'
    | 'GRANDMASTER'
    | 'IRON'
    | 'MASTER'
    | 'PLATINUM'
    | 'SILVER';

export type SummonerPastRank = {
    season: string;
    tier: Tier;
    rank: number;
    lp: number;
};

export type SummonerRanking = {
    ranking: number;
    percentage: number;
};
eddy-kor-92 commented 1 year ago

@minyoungyang

export type SummonerRanking = {
    rank: number;
    percentage: number;
};

여기서의 rank는 랭킹 순위를 표현하는 숫자이죠?

그리고

export type SummonerPastRank = {
    season: string;
    tier: Tier;
    rank: number;
    lp: number;
};

여기서의 rank는 bronze 1 같이 표현할 때 bronze가 tier, 1이 rank라고도 하는데 이 때의 rank를 표현하는 숫자이죠?

여기까지 제가 이해한게 맞다면 둘 중 하나의 네이밍을 바꿔야 할 듯 합니다. (혼선이 생김)

추가로 bronze 1과 같이 소환사의 티어정보는 TIER + RANK로 이루어져 있기 때문에 currentTier 뿐만 아니라 currentRank도 있어야 할 것 같아요

minyoungyang commented 1 year ago
export type SummonerRanking = {
    ranking: number;
    percentage: number;
};

으로 변경

minyoungyang commented 1 year ago

진행중