Skip to content

Conversation

@Orwiss
Copy link
Contributor

@Orwiss Orwiss commented Apr 8, 2024

Changes: Translate access.md

PR Checklist

  • npm run lint passes
  • [Inline documentation] is included / updated
  • [Unit tests] are included / updated

@Orwiss
Copy link
Contributor Author

Orwiss commented Apr 8, 2024

@designerSejinOH @yinhwa PR 완료 했습니다!

@designerSejinOH designerSejinOH self-requested a review April 8, 2024 12:40
@designerSejinOH
Copy link
Contributor

@Orwiss 빠른 과제 수행 감사드립니다! 곧 제가 1차 리뷰 진행하도록 하겠습니다! @yinhwa

Copy link
Contributor

@designerSejinOH designerSejinOH left a comment

Choose a reason for hiding this comment

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

@Orwiss 1차 리뷰 완료하였습니다! (22:40-23:25, 45분) 다소 어려운 내용이나 디테일하게 잘 번역해주셔서 감사합니다..! 몇 가지 코멘트만 남겨두었습니다! @yinhwa 2차 리뷰 바로 진행해주시면 될 것 같습니다!

@Qianqianye Qianqianye added the Translation Translation of p5.js documentation label Apr 8, 2024
@yinhwa
Copy link
Contributor

yinhwa commented Apr 9, 2024

@Orwiss 성훈님 1차리뷰 코멘트 반영해주시면 2차 리뷰 이어서 진행하겠습니다 :) 화팅팅💥💥 @designerSejinOH

first check
@Orwiss
Copy link
Contributor Author

Orwiss commented Apr 9, 2024

@designerSejinOH @yinhwa 1차 리뷰 코멘트 반영 완료 했습니다~!!

Copy link
Contributor

@yinhwa yinhwa left a comment

Choose a reason for hiding this comment

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

@Orwiss 성훈님! p5.js 커뮤니티의 가장 중요한 문서를 이렇게 멋지게 번역해주셔서 감사드립니다 ㅎㅎ 2차 리뷰 완료 (4:00-4:40, 총 40분)하였고 코멘트 남겨두었습니다!

다른 기여자님들께서도 이 문서 번역문 한번 읽어보시면 좋을것 같아 멘션드립니다 💜@yunyoungJang @YewonCALLI @ienground @designerSejinOH

@Orwiss
Copy link
Contributor Author

Orwiss commented Apr 9, 2024

@yinhwa 2차 리뷰 반영 완료했습니다!!

@yinhwa
Copy link
Contributor

yinhwa commented Apr 9, 2024

@Orwiss 성훈님 빠른 작업감사드립니다! 내일 오전 중에 마이너한 문법만 제가 직접 수정후 승인 완료하겠습니다 :) @designerSejinOH

#접근성에 대한 초점 과 ##접근성의 종류 에서 몇가지 표현만 바꾸어 파이널라이즈합니다! @Orwiss
Copy link
Contributor

@yinhwa yinhwa left a comment

Choose a reason for hiding this comment

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

Hi, @Qianqianye @limzykenneth we're ready to merge this PR :) thank you @Orwiss @designerSejinOH !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Translation Translation of p5.js documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants