Conversation
- 피드백 링크는 나오면 추가할 예정
- 구현되어있지 않으므로 주석처리
| --color-background-neutral-subtle: var(--color-gray-200); | ||
| --color-background-neutral-default: var(--color-gray-500); | ||
| --color-background-neutral-strong: var(--color-gray-900); |
There was a problem hiding this comment.
이것들은 혹시 새로 디자인 시스템에 추가된 건가요??
만약 이번에 추가된 게 아니라 원래 있던 건데 등록되어 있지 않았던거라면, 디자인 시스템도 나중에 한 번 쭉 수정해야 하긴 해야겠네요....😥
There was a problem hiding this comment.
원래 디자인에는 있었는데, 이 파일에 없어서 추가했습니다!
There was a problem hiding this comment.
혹시 이건 뭔가요....?
이미지가 보이지 않아서...어디에 쓰는 녀석인가요?
There was a problem hiding this comment.
혹시 이 파일 한 번만 확인해주실 수 있나요?
원래 my-page에 있던 녀석인데, 아무래도 유저 관련으로 공통적으로 쓰이게 될 UI 같아서 entity로 옮겼습니다...
이 과정에서 좀 문제가 있던 건지 feature과 entity 두 곳에 모달이 존재하는 것 같아요
There was a problem hiding this comment.
두 곳에 있네요! feature에 있는 모달 삭제하겠습니다.
| export interface UserPositiveKeywordsResponse { | ||
| totalCount: number | null; // params에 pageSize 값을 보내지 않는 경우 null | ||
| reviewerCount: number | null; // params에 pageSize 값을 보내지 않는 경우 null | ||
| keywords: Keyword[]; | ||
| } |
There was a problem hiding this comment.
이거 저도 좀 고민했던 방식인데 저희가 null 값을 보낼 경우 통합 형식을 한 번 정하면 좋을 것 같아요.
totalCount: number | null; 이런 식으로 null을 명시해주면 이게 null을 포함하는 걸 한 눈에 알 수 있어서 좋긴 한데
저는 조금 긴가? 싶어서 totalCount?: number; 이런 형식으로 통합해서 작성했었거든요.
두 방법 중에 저희가 편한 방식 정해서 그걸로 전체 통일하면 좋을 것 같습니다!
ㅋㅋㅋㅋ 아무래도 저희 둘 다 작업이 섞이다보니 이게.......헷갈릴 수 밖에 없는 것 같습니다....🤦 |
2 participants
☘️ 작업 내용
민주님... 저도 #121 이 작업과 스터디 후기 작업이 합쳐졌습니다...
죄송합니다 🙇
코드 양이 많아서 미리 올립니다.
구현이 안된 부분부터 설명드릴게요. 스터디 후기 기능 자체는 지금 현재 테스트하기 어려우실 수 있는데요.
(평일에는 잘 조회되던 api에서 에러 응답이 와서, 여쭤본 상태입니다.)
1. 스터디 후기 모달 렌더링
2. 후기 작성 완료 모달