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.

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

플레이코딩박스 하드웨어 블록 #2882

Open
wants to merge 1 commit into
base: develop-hw
Choose a base branch
from

Conversation

piaojongwien
Copy link

플레이코딩박스 하드웨어 블록 추가

  1. block_playcodingbox.js 파일 추가:
  2. 디지털 및 아날로그 핀 설정.
  3. RGB LED, 부저, 서보 모터 등 주요 액추에이터 제어 블록 구현.
  4. 미사용 핀 정보:
    -가변저항(A6): 사용하지 않음.
    -조도센서(A7): 사용하지 않음.
  5. 추후 업데이트 예정:
  • 도트매트릭스 관련 블록: 추후 추가 예정.

@Tnks2U Tnks2U changed the base branch from develop to develop-hw December 19, 2024 05:49
@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 19, 2024

@piaojongwien
타깃 브랜치는 반드시 entrylabs:develop-hw이어야 합니다. 유의 부탁드립니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 19, 2024

@piaojongwien
108번 라인의 name값은 'Playcodingbox'로 되어 있지만, isNotFor값은 'playcodingbox'로 되어 있습니다.(첫 알파벳 대소문자)
두 값은 완전히 같아야, 연결되었을때 블록이 노출됩니다. 수정 부탁드립니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 19, 2024

@piaojongwien
일정상 내일(금) 오후 2시까지 PR이 가능합니다. 이후부터는 reject하고 이번달 배포에서 제외됩니다. 유의 부탁드립니다.

@piaojongwien
Copy link
Author

요청하신 name 값과 isNotFor 값의 대소문자 불일치를 수정하였습니다.

  • 수정 전: name: 'Playcodingbox', isNotFor: ['playcodingbox']
  • 수정 후: name: 'playcodingbox', isNotFor: ['playcodingbox']

다시 확인 부탁드립니다. 감사합니다!

@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 19, 2024

@piaojongwien
https://github.com/entrylabs/entryjs/pull/2882/files
file changes 탭에서 여전히 수정되지 않은 것으로 나타납니다.
한 번 더 확인 부탁드립니다.

@piaojongwien
Copy link
Author

확인부탁드립니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 20, 2024

@piaojongwien
name값과 isNotFor 불일치 문제는 해결되었지만, 공용파일인 pnpm-lock.yml 파일이 수정되었습니다.
공용파일은 수정되어선 안됩니다. file changes 탭을 반드시 확인하시기 바랍니다.

https://github.com/entrylabs/entryjs/pull/2882/files#diff-32824c984905bb02bc7ffcef96a77addd1f1602cff71a11fbbfdd7f53ee026bb

@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 20, 2024

@piaojongwien
30분 이후부터 빌드에 들어가야 해서, 확인 부탁드립니다.
이후부터는 반영이 어렵습니다.
entry-hw쪽 PR에도 문제가 있으니 함께 확인 부탁드립니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Dec 20, 2024

@piaojongwien
추가 작업이 없으셔서 공지드린대로 이번달 배포에선 제외됩니다.
요청드린 내용 작업주시면 다음달 배포에 반영하겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants