Skip to content

10월 FE 블로그 포스팅 #236

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
Merged
merged 7 commits into from
Nov 10, 2022
Merged

10월 FE 블로그 포스팅 #236

merged 7 commits into from
Nov 10, 2022

Conversation

@baebae02
Copy link
Member

@baebae02 baebae02 commented Oct 21, 2022

save-data-without-server에 대해서 포스팅해보았습니다.
리뷰 부탁드립니다!

poiu694
poiu694 reviewed Oct 22, 2022
View reviewed changes
Copy link

@poiu694 poiu694 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 웹스토리지는 짱짱 간편한 친구라 정말 좋은 것 같아요! 좋은 글 감사합니다 👍

저는 지킬을 실행 시켜보진 않지만, 깃허브 에서 강조 표시 내 따옴표나, 사진같은게 첨부가 잘 안되어있는 것 같아요!

블로그 분들께서도 프리뷰로 확인해주시면 좋을 것 같습니당 :)

@baebae02
Copy link
Member Author

baebae02 commented Oct 22, 2022

@poiu694 리뷰 해주신 것과 이미지 경로 문제까지 수정했습니다! 세심한 리뷰 정말 감사합니다 :)

@SSUHYUNKIM SSUHYUNKIM added the 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ label Oct 22, 2022
pasly0920
pasly0920 reviewed Oct 22, 2022
View reviewed changes
Copy link
Member

@pasly0920 pasly0920 left a comment

Choose a reason for hiding this comment

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

전반적으로 물흐르듯이 읽혀서 좋은 글이라고 생각됩니다. 좋은 글 감사드리고 고생많으셨습니다

@baebae02
Copy link
Member Author

baebae02 commented Oct 23, 2022

@pasly0920 리뷰 반영했습니다 :)

pasly0920
pasly0920 previously approved these changes Oct 23, 2022
View reviewed changes
Copy link
Member

@pasly0920 pasly0920 left a comment

Choose a reason for hiding this comment

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

LGTM!

poiu694
poiu694 previously approved these changes Oct 23, 2022
View reviewed changes
@poiu694 poiu694 added ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 and removed 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Oct 23, 2022
@SSUHYUNKIM
Copy link

SSUHYUNKIM commented Nov 9, 2022 •
edited
Loading

서현

  1. 혹시 예상했던 부분과 다르게 들어간 부분이 있으면 수정해주세요.
  2. 모든 수정 후 맞춤법 검사 한번 부탁드립니다.

서현 - markdown 문법 문제
서현 - markdown 문법 문제2

  1. 아마 이모티콘 때문에 오류가 생기는 것 같은데 수정 부탁드립니다.

@SSUHYUNKIM SSUHYUNKIM added the 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 label Nov 9, 2022
@baebae02 baebae02 dismissed stale reviews from poiu694 and pasly0920 via 631171d November 10, 2022 14:25
@baebae02
Copy link
Member Author

baebae02 commented Nov 10, 2022

@SSUHYUNKIM 이모티콘 에러, 맞춤법 검사 완료하였습니다 !

@SSUHYUNKIM SSUHYUNKIM added 💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때 and removed ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 labels Nov 10, 2022
@SSUHYUNKIM SSUHYUNKIM merged commit ea42be2 into GDSC-University-of-Seoul:master Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@poiu694 poiu694 poiu694 left review comments

@pasly0920 pasly0920 pasly0920 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@baebae02 baebae02

Labels

💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@baebae02 @SSUHYUNKIM @poiu694 @pasly0920