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

[GGFE-129] readme 업데이트 #895

Merged
merged 5 commits into from
Jun 30, 2023
Merged

[GGFE-129] readme 업데이트 #895

merged 5 commits into from
Jun 30, 2023

Conversation

yoouyeon
Copy link
Member

@yoouyeon yoouyeon commented Jun 28, 2023 •
edited
Loading

📌 개요

  • 3기 정보, 이번에 업데이트 된 내용에 대한 내용 추가

💻 작업사항

  • 상단 로고 사진을 favicon 이미지로 바꾸었는데 괜찮은지 모르겠네요...
  • 3기 정보 추가했습니다. 정보 제대로 적혀 있는지 확인해주세요!
  • 업데이트 된 디자인이랑 기능에 대한 스크린샷이랑 설명 추가했습니다. 기존에 적혀있던 설명에는 말투(?)가 좀 섞여 있어서 어색한거같기도 한데 어떤 식으로 적는게 좋을지 모르겠네요...
  • 이 외에도 읽어보시고 뭔가 추가하거나 수정하고 싶은 내용 있으시면 말씀해주세용

✅ 변경로직

  • 상단 로고 사진을 favicon 이미지 + 글자 로고 이미지로 넣었는데 두개 같이 넣으니까 제가 보기엔 좀 어색한 것 같습니다.. 둘 중 하나만 넣거나 다른 방법으로 넣는게 좋을 것 같아요..;;

@yoouyeon yoouyeon self-assigned this Jun 28, 2023
PHJoon
PHJoon approved these changes Jun 28, 2023
View reviewed changes
Copy link
Contributor

@PHJoon PHJoon left a comment •
edited
Loading

Choose a reason for hiding this comment

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

상황별로 스크린샷이랑 설명도 잘 써져있고 다른 부분들도 잘 수정됐네요! 고생하셨습니다!! 👍👍
mosong님 리뷰보고 다시 보니까 프로필페이지에서 kakao 연동하기도 추가할 수 있을거 같네요

mike2ox
mike2ox reviewed Jun 29, 2023
View reviewed changes
Copy link
Contributor

@mike2ox mike2ox left a comment

Choose a reason for hiding this comment

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

그리고 kakao OAuth 붙였으니 42카뎃이 아닌 일반인은 어디까지 이용이 가능한지, 42gg 정책관련해서도 써주거나 정책 페이지 링크 제공해주는게 좋아보여요

고생하셨습니다 👍

README.md
@@ -26,14 +26,17 @@
<img src="https://img.shields.io/badge/Recoil-3578E5?style=for-the-badge&logo=&logoColor=white"/>
<img src="https://img.shields.io/badge/MUI-%230081CB.svg?style=for-the-badge&logo=mui&logoColor=white"/>

<img src="https://img.shields.io/badge/Storybook-FF4785.svg?style=for-the-badge&logo=storybook&logoColor=white"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

스토리북도 외부에서 볼 수 있도록 설정 & 안내해주면 좋을거 같아요

Copy link
Member Author

Choose a reason for hiding this comment

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

으악... 사실 스토리북은 진짜 개발 중 테스트용으로 만든거긴 한데,,, 배포도 고민해봐야겠네요.... 😥 의견감사합니닷

README.md Outdated
@@ -2,7 +2,7 @@

<div align=center>

<img width="600" alt="Frame 3" src="https://user-images.githubusercontent.com/86540825/209137349-57356fde-777f-4d4f-877f-b5f2b2053da0.png">
<img width="10%" alt="42GG_Logo" src="https://github.com/42organization/42gg.client/assets/57761286/210c65a0-5e81-4b35-860d-7107442b987c">
Copy link
Contributor

Choose a reason for hiding this comment

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

로고 옆에 42gg 메인 글자 로고도 넣는게 좋아보여요

@42sungwook
Copy link
Contributor

오 고생하셨습니다. 저도 개인적으로는 42gg 네온 로고보다 그냥 42GG 글씨 넣는게 더 이쁜 것 같아요

parksangmin1543
parksangmin1543 approved these changes Jun 30, 2023
View reviewed changes
Copy link
Contributor

@parksangmin1543 parksangmin1543 left a comment

Choose a reason for hiding this comment

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

변경사항 확인했습니다.

@42sungwook 42sungwook merged commit 903cd93 into main Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mike2ox mike2ox mike2ox left review comments

@parksangmin1543 parksangmin1543 parksangmin1543 approved these changes

@PHJoon PHJoon PHJoon approved these changes

Assignees

@yoouyeon yoouyeon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yoouyeon @42sungwook @mike2ox @PHJoon @parksangmin1543