Skip to content

Conversation

@Crocuse
Copy link
Contributor

@Crocuse Crocuse commented Aug 24, 2023

유의사항

  • base Branch 는 develop-hw 로 통일하였습니다. master, develop 이 아닙니다.
  • 체크박스의 체크는 이슈 생성 후 직접 클릭 or [x] 와 같이 체크하면 됩니다.

어떤 하드웨어 인가요?

[하드웨어명 / 회사명]
ProboConnect / Probo

어떤 종류의 PR 인가요?

  • 기존 하드웨어 변경

이 PR 에서 어떤 부분이 변경되었나요? (작성필요)

  • 펌웨어 설치 업데이트

PR 에 대해 아래 부분을 확인해주세요

  • 코드 문법 오류가 발생하진 않았는가
  • 코딩 컨벤션을 정리했는가 (린트, 들여쓰기, 등등..)
  • base branch 설정을 develop-hw 으로 했는가?
  • PR 생성 후, 기존 코드와 충돌이 나진 않았나요?

@Tnks2U
Copy link
Contributor

Tnks2U commented Sep 12, 2023

@Crocuse
package-lock.json이 변경사항에 포함되어 있습니다. 해당 변경사항을 제외해 주시기 바랍니다.

추가로, 커밋에 이상한 부분이 있어 문의드립니다.

  • 커스텀 버튼 관련 로직이 추가되었는데, 커스텀 버튼의 동작은 단순히 콘솔만 찍는 내용입니다. 의도하신 기능이 맞을까요?
  • json파일에서 '드라이버 설치' 가 '펌웨어 설치'로 문구가 변경되었습니다. 올바른 문구인 '드라이버 설치'로 변경 부탁드립니다.
  • js파일에서 Lodash를 require로 불러오고 있습니다만, 사용하는 부분이 없습니다. 불필요한 모듈추가는 제거를 부탁드립니다. 또한, package.json에 포함되지 않은 모듈을 파일에서 직접 가져오면 오류가 발생할 수 있습니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Sep 19, 2023

@Crocuse
pakcage.json이 제외된 것은 확인했습니다만, 나머지 요청사항이 반영되지 않아서 merge후 문제가 있을 경우 반영이 어려울 수 있습니다. 양해 부탁드립니다. (테스트중 문제가 없으면 정상 반영됩니다.)

@Tnks2U Tnks2U merged commit eea045b into entrylabs:develop-hw Sep 19, 2023
@Tnks2U
Copy link
Contributor

Tnks2U commented Sep 19, 2023

@Crocuse
펌웨어 다운로드로 설치파일을 받은 후 전용 프로그램으로 프로보 커넥트에 펌웨어를 업데이트 했지만, 연결이 되지 않습니다.
proboConnect.json에서 selecPort 옵션을 "selectPort": true 로 되돌리고 정상연결을 확인했습니다.
추가된 블럭중 서보모터처럼 제공해주신 샘플에 없는 모듈은 테스트하지 못했습니다.

변경된 코드사항은 아래와 같습니다. 문제가 있다면 코멘트 부탁드립니다.
9416688

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