Skip to content

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

Merged
merged 15 commits into from
Jun 22, 2021
Merged

Feature/testing second add #86

merged 15 commits into from
Jun 22, 2021

Conversation

beckyi
Copy link
Contributor

@beckyi beckyi commented Jun 15, 2021

테스팅 2차 개발

중간 발표 이후 미개발된 요소들 개발을 시작하자...!
: 댓글, 결과 위주로 기능 추가합니다

  • 댓글 수정, 삭제 (API 필요)

  • 메이커 한마디

  • 회원 가입 유도 넛징

  • 회원가입 후 테스트 결과 저장
    추가적으로 수정합니다.

  • figma design css 적용

  • 버튼명 대문자로 변환되는 이슈 처리

@beckyi beckyi self-assigned this Jun 15, 2021
@beckyi beckyi added the testing testing label Jun 15, 2021
@beckyi
Copy link
Contributor Author

beckyi commented Jun 15, 2021

댓글 관련 API가 아직 나와 있지 않는 관계로 로직만 개발 (로그인 유저 확인 등 유효성 검사 포함)

@1000peach
Copy link
Member

테스트를 한번 진행하면 다시 테스트를 누르거나 다른 페이지 이동 시
전에 했던 데이터가 남아서 잠깐 보이는 현상이 있어요! 깜빡깜빡
이 부분도 그냥 아무것도 없으면 흰 화면 띄우거나 로딩 컴포넌트 띄우면 더 좋은 ux가 나올 거 같습니당. 👍
화이태잉!!!

@1000peach
Copy link
Member

그 비공개 테스트가 두 가지 버전이라는 게, 저는 이렇게 생각했는데

  1. 비공개 테스트를 피드에 노출하지 않음 -> 피드에만 노출되지 않고 테스트 자체는 링크로 접근 가능.
  2. 테스트 자체를 아예 비공개 -> 피드에 노출하지 않고, 메이커만 볼 수 있음. 메이커만 링크로 직접 접근 가능

생각해보니까 2번 기능은 없는 거 같아요 ㅎㅎㅎ 🤔 1번 처리만 될 듯 합니다! 그냥.. 넘겨주세요 (?) 하하

@beckyi
Copy link
Contributor Author

beckyi commented Jun 16, 2021

  1. 회원 가입 유도 넛징, 테스트 평가는 로그인 완료 시 api 호출하도록 구현할 예정
  2. 메이커 한마디 (API 연동까지 완료되었지만 emoji는 아직 파라미터 값이 미정되어 보류)

@beckyi
Copy link
Contributor Author

beckyi commented Jun 16, 2021

테스트를 한번 진행하면 다시 테스트를 누르거나 다른 페이지 이동 시
전에 했던 데이터가 남아서 잠깐 보이는 현상이 있어요! 깜빡깜빡
이 부분도 그냥 아무것도 없으면 흰 화면 띄우거나 로딩 컴포넌트 띄우면 더 좋은 ux가 나올 거 같습니당. 👍
화이태잉!!!

데이터가 남아있다는 이슈 확인했습니다 :)

데이터 있을 경우 한번 초기화 해줄게요~!

@1000peach
Copy link
Member

1000peach commented Jun 16, 2021

오오 빠른 QA와 이슈 해결 멋져요.. 저도 쫌쫌따리 확인하고 있습니다.. 케케
댓글 css 변경한 건 내일 점심 전에 올려볼게요!

올렸어요! #89 🙆‍♀️

@@ -94,7 +124,12 @@ const Result = memo((props) => {
<TitleBox>
<RankingList top={5} userRanking={rankOrder} noline />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

카톡에 남겼어요! 백엔드에서 주면 top 바꿔주면 될 거 같아요 카톡에는 묻혀서 모두 까먹을까봐 메모를..! ✍

@beckyi beckyi linked an issue Jun 19, 2021 that may be closed by this pull request
7 tasks
@1000peach
Copy link
Member

피그마

image

개발 화면

image

테스트 결과 부분에서 가장 많은 유형처럼 제목들 line-height 빠진 거 같아요!
line-height: 3.6rem 입니당

@1000peach
Copy link
Member

테스트 메이킹 만들 때 option에 영어 소문자로 입력하면 출력할 때 무조건 대문자로 나와요!

선택지만 그런 거 같아요.! 나머지는 다 소문자로 잘 출력돼여
network로는 소문자로 잘 오는데, 대문자로 바뀌는 걸 보니 material Button에서 대문자로 바꾸는 거 같기도 합니다

image

@beckyi
Copy link
Contributor Author

beckyi commented Jun 21, 2021

피그마

image

개발 화면

image

테스트 결과 부분에서 가장 많은 유형처럼 제목들 line-height 빠진 거 같아요!
line-height: 3.6rem 입니당

가장 많은 유형 높이 수정햇습니다.

@beckyi beckyi closed this Jun 21, 2021
@beckyi beckyi reopened this Jun 21, 2021
@rktguswjd
Copy link
Member


열씸히 해봅시다

@beckyi
Copy link
Contributor Author

beckyi commented Jun 22, 2021

회원 가입 유도 넛징 얼럿은 공통 NoticeAler 소스 develop에 적용될 때 수정하겠습니다.

그 외 테스팅 추가 작업은 API 및 공동작업 필요하므로 다른 브랜치로 생성해서 적용하겠습니다.
총 처리 사항은 이슈에 기록하겠습니다.

@1000peach @rktguswjd

@beckyi beckyi merged commit 52ef47a into develop Jun 22, 2021
@beckyi beckyi deleted the feature/Testing-second-add branch June 22, 2021 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Testing-1st-supplement
3 participants