Skip to content
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

[test] 서울콘 대응 #2966

Closed
wants to merge 22 commits into from
Closed

[test] 서울콘 대응 #2966

wants to merge 22 commits into from

Conversation

seulgiyoon
Copy link
Contributor

PR 설명

서울콘에서 보여주는 영역과 로직 분기

  1. Triple document에 guestMode를 추가, poi list에서 스크랩 하트 아이콘을 숨김
    guestMode는 로그인 관련 영역과 로직, App으로의 연결을 유도하는 영역과 로직을 숨기는 flag
  2. ImageCarousel에 guestMode or readonly 등의 prop 전달 가능하도록 수정
    위의 guestMode와 마찬가지로, 해당 플래그를 기준으로 사진 없을 때 표시방식을 분기하고 사진을 5장까지만 노출

서울콘에서 영문 데이터만 보여주기

  1. Poi list에서 이름과 지역명을 영어로 보여주기
  2. Static Map에서 language를 prop으로 전달
    구글 static map에 language 쿼리 전달 가능하도록

- '서울콘' 행사 대응용
- '서울콘' 행사를 통해 triple-content-web의 페이지로 유입된 사용자들은 모두 비로그인 상태이며, 로그인이 필요한 기능을 사용할 수 없음. 더불어 트리플앱으로의 연결을 유도하는 루트도 보지 않아야 함
- Triple Document로 제공되는 가이드와 POI 영역에서 로그인 및 앱과의 연결과 관련된 부분들을 guestMode라는 값을 기준으로 분기하기 위하여 값을 추가(예 : guestMode가 true인 경우, Pois 컴포넌트의 스크랩 버튼 숨기기)
- extended-poi-list-element는 Pois 엘리먼트에서 사용되지 않으나, poi-list-elements에서의 사용성 통일을 위해 다른 poi element들과 동일하게 숨김 로직 반영
- '서울콘' 행사 대응용
- '서울콘' 행사를 통해 triple-content-web의 페이지로 유입된 사용자들은 모두 비로그인 상태이며, 로그인이 필요한 기능을 사용할 수 없음. 더불어 트리플앱으로의 연결을 유도하는 루트도 보지 않아야 함
-  image-carousel에서 로그인 및 앱과의 연결과 관련된 부분들을 guestMode라는 값을 기준으로 분기하기 위하여 값을 추가(예 : guestMode가 true인 경우, 사진이 5장 이상일 때 '트리플 앱에서 보기' 노출로직 숨김)
- 5장째 사진에도 인덱스를 보여줌
- 사진 총 장수를 최대 5까지만 표기
@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Attention: 76 lines in your changes are missing coverage. Please review.

Comparison is base (b7a7a46) 12.17% compared to head (39ccf42) 12.17%.
Report is 23 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2966      +/-   ##
==========================================
- Coverage   12.17%   12.17%   -0.01%     
==========================================
  Files         687      688       +1     
  Lines       36212    36229      +17     
  Branches     8941     8947       +6     
==========================================
  Hits         4410     4410              
- Misses      31795    31812      +17     
  Partials        7        7              
Files Coverage Δ
packages/triple-document/src/elements/pois.tsx 0.00% <0.00%> (ø)
packages/poi-detail/src/image-carousel/index.tsx 0.00% <0.00%> (ø)
...ackages/poi-list-elements/src/carousel-element.tsx 0.00% <0.00%> (ø)
...ges/triple-document/src/prop-context/guest-mode.ts 0.00% <0.00%> (ø)
packages/triple-document/src/triple-document.tsx 0.00% <0.00%> (ø)
...poi-detail/src/image-carousel/carousel-section.tsx 0.00% <0.00%> (ø)
...ackages/poi-detail/src/image-carousel/carousel.tsx 0.00% <0.00%> (ø)
...poi-list-elements/src/compact-poi-list-element.tsx 0.00% <0.00%> (ø)
...ages/poi-detail/src/image-carousel/placeholder.tsx 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@seulgiyoon
Copy link
Contributor Author

release-canary

@github-actions
Copy link

v13.8.2-pr-2966.14 has been published!

@seulgiyoon
Copy link
Contributor Author

release-canary

@github-actions
Copy link

v13.8.2-pr-2966.19 has been published!

@seulgiyoon
Copy link
Contributor Author

release-canary

Copy link

github-actions bot commented Nov 7, 2023

v13.8.2-pr-2966.24 has been published!

@seulgiyoon seulgiyoon closed this Nov 10, 2023
@seulgiyoon seulgiyoon deleted the test/seoul-con-and-i18n branch November 10, 2023 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant