-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/testing second add #86
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
Conversation
댓글 관련 API가 아직 나와 있지 않는 관계로 로직만 개발 (로그인 유저 확인 등 유효성 검사 포함) |
테스트를 한번 진행하면 다시 테스트를 누르거나 다른 페이지 이동 시 |
그 비공개 테스트가 두 가지 버전이라는 게, 저는 이렇게 생각했는데
생각해보니까 2번 기능은 없는 거 같아요 ㅎㅎㅎ 🤔 1번 처리만 될 듯 합니다! 그냥.. 넘겨주세요 (?) 하하 |
|
데이터 있을 경우 한번 초기화 해줄게요~! |
오오 빠른 QA와 이슈 해결 멋져요.. 저도 쫌쫌따리 확인하고 있습니다.. 케케 올렸어요! #89 🙆♀️ |
[fix] 댓글 영역 배경 고정
@@ -94,7 +124,12 @@ const Result = memo((props) => { | |||
<TitleBox> | |||
<RankingList top={5} userRanking={rankOrder} noline /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
카톡에 남겼어요! 백엔드에서 주면 top
바꿔주면 될 거 같아요 카톡에는 묻혀서 모두 까먹을까봐 메모를..! ✍
넵 |
회원 가입 유도 넛징 얼럿은 공통 NoticeAler 소스 develop에 적용될 때 수정하겠습니다. 그 외 테스팅 추가 작업은 API 및 공동작업 필요하므로 다른 브랜치로 생성해서 적용하겠습니다. |
테스팅 2차 개발
중간 발표 이후 미개발된 요소들 개발을 시작하자...!
: 댓글, 결과 위주로 기능 추가합니다
댓글 수정, 삭제 (API 필요)
메이커 한마디
회원 가입 유도 넛징
회원가입 후 테스트 결과 저장
추가적으로 수정합니다.
figma design css 적용
버튼명 대문자로 변환되는 이슈 처리