Skip to content

캐릭터 API 연결과 디자인 적용 #110

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 6 commits into from
Feb 22, 2025

Conversation

yoouyeon
Copy link
Collaborator

📝 관련 이슈

💻 작업 내용

1. ⭐️ 받을 정산 헤더 부분을 조금 수정했습니다!!!

1.1 배경색 버그 수정

DescriptionField의 기본 색깔을 흰색으로 설정하고 props로 색깔을 전달해서 회색으로 설정할 수 있도록 해뒀더니 이 페이지에 영향을 끼쳤더라구요.. 회색이 적용되도록 다시 수정했습니다.

1.2 계좌번호 복사 포맷 변경

지금은 계좌번호만 복사되게 되어 있는데 보다보니 은행도 같이 복사되어야 송금앱에서 인식을 해줬던 것 같아서 신한 123456789 요런 식으로 복사와 표시 포맷을 바꿔주었습니다.
혹시 잘못된 기능이면 말씀주세요!! 당장 복구하겠습니당

2. 캐릭터 공유 API 연결과 화면 구성

2.1 캐릭터 공유 API를 연결하고 엉망이던 배경색을 정리했습니다

2.2 데이터가 없는 경우의 페이지를 추가했습니다.

지금은 정산이 완료되지 않은 경우에도 api를 호출하면 캐릭터 데이터가 잘 오는 상황이긴 한데요 (....)
이 페이지는 임의로 접근 가능한 페이지이기도 해서 null로 두면 안될 것 같아 못생긴(...) 에러 페이지를 하나 만들어두었습니다.
(API가 변경될수도 있으니..?)

📸 스크린샷

Screen_Shot 2025-02-22 04 41 27 Screen_Shot 2025-02-22 04 41 08 Screen_Shot 2025-02-22 04 59 06

@yoouyeon yoouyeon self-assigned this Feb 21, 2025
@yoouyeon yoouyeon linked an issue Feb 21, 2025 that may be closed by this pull request
3 tasks
@github-actions github-actions bot requested a review from ongheong February 21, 2025 20:00
@github-actions github-actions bot added ✨ Feature 새로운 기능 추가나 개선을 위한 작업 🖥️ FE 프론트엔드 작업 labels Feb 21, 2025
Copy link
Member

@ongheong ongheong left a comment

Choose a reason for hiding this comment

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

오 계좌 복사에 대해서는 바뀐 기능이 더 적합한 것 같습니다! 에러 페이지까지 만들어주셨군요... 좋습니당

@yoouyeon yoouyeon merged commit abcfd3b into develop Feb 22, 2025
4 checks passed
@yoouyeon yoouyeon deleted the feat/#106-image-character-bank branch February 22, 2025 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖥️ FE 프론트엔드 작업 ✨ Feature 새로운 기능 추가나 개선을 위한 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

이미지 작업 (캐릭터, 은행)
2 participants