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.

Sign up for GitHub

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

Jump to bottom

[비즈니스] 코인 사장님 아이콘 적용 #361

Merged
merged 2 commits into from
Jun 25, 2024
Merged

[비즈니스] 코인 사장님 아이콘 적용 #361

merged 2 commits into from
Jun 25, 2024

Conversation

chaeseungyun
Copy link
Contributor

  • Close #ISSUE_NUMBER

What is this PR? 🔍

  • 기능 :
  • issue : #

Changes 📝

새로운 코인 사장님 아이콘을 적용하고 웹 이름을 코인 사장님으로 수정했습니다.
헤더 아이콘은 따로 없어서 유지해야할 것 같습니다

ScreenShot 📷

image
image

Test CheckList ✅

  • test 1
  • test 2
  • test 3

Precaution

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

@chaeseungyun chaeseungyun self-assigned this Jun 25, 2024
dooohun
dooohun approved these changes Jun 25, 2024
View reviewed changes
Copy link
Contributor

@dooohun dooohun left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@chaeseungyun chaeseungyun merged commit 9ec2615 into develop Jun 25, 2024
2 checks passed
@chaeseungyun chaeseungyun deleted the feat/apply_new_icon branch June 25, 2024 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dooohun dooohun dooohun approved these changes

@kimeodml kimeodml Awaiting requested review from kimeodml

Assignees

@chaeseungyun chaeseungyun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chaeseungyun @dooohun