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

프로모션 눌렀을 때 UI 변경 / + 인코딩 이슈 해결 / DesignSystem의 primary 색상명 변경 #80

Merged
merged 4 commits into from
Mar 4, 2024

Conversation

WhiteHyun
Copy link
Member

Screenshots 📸

적용 영상
RPReplay_Final1709536612 2



고민, 과정, 근거 💬

네트워크 인코딩

+기호를 URL에서 사용하기 위해 %2B로 변경했습니다.[1] +는 URL에서 직접 사용할 수 없어서 UTF-8문자로 변환해야합니다.
URLQueryItem을 생성할 때 같이 인코딩해서 넣어주면 될 줄 알았지만, request에 url을 넣을 때 로깅을 확인한 결과, %2B에서 %자체를 또 UTF-8로 변환해서 최종적으로 %252B 가 들어가는 현상을 발견했습니다. 그래서 마지막에 request.url을 초기화하기 전에, +기호를 인코딩한 URL String 가지고 URL을 다시 새롭게 생성하는 방법으로 해결했습니다.

UI 수정

SwiftUI에서도 primary라는 색상이 존재하기에, DesignSystem과 겹치는 경우가 발생했습니다. 그래서 기존 DesignSystem의 primary색상을 pyeonHaengPrimary로 변경했습니다.
최대한 PyeonHaeng명을 안쓰고 싶지만 ㅎㅎ.. 이거 외에 떠오르는 변수명이 없었습니다.



References 📋

  1. HTML URL Encoding Reference




@WhiteHyun WhiteHyun added 🏠 Home Home View 🔨 Implementation New feature or request labels Mar 4, 2024
@WhiteHyun WhiteHyun added this to the v2.0.0 milestone Mar 4, 2024
@WhiteHyun WhiteHyun requested a review from a team March 4, 2024 07:27
@WhiteHyun WhiteHyun self-assigned this Mar 4, 2024
Copy link
Contributor

@eung7 eung7 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !

@eung7 eung7 merged commit bb7b3e5 into main Mar 4, 2024
6 checks passed
@eung7 eung7 deleted the feature/home/79 branch March 4, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏠 Home Home View 🔨 Implementation New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

기본값인 All이 아닌 경우 할인 행사 버튼의 바탕화면을 메인 컬러로 변경한다.
2 participants