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

에이럭스 테크닉 파워, 일반 신규 등록 #778

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

hakueon
Copy link
Contributor

@hakueon hakueon commented Oct 16, 2024

유의사항

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

어떤 하드웨어 인가요?

[테크닉 파워 / 에이럭스]
[테크닉 일반 / 에이럭스]

어떤 종류의 PR 인가요?

  • 신규 하드웨어 추가
  • 기존 하드웨어 변경
  • 기타

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

  • 테크닉 파워, 일반 공용 함수 관련 파일 추가
  • 테크닉 파워 관련 파일 추가
  • 테크닉 일반 관련 파일 추가
  • 테크닉 파워, 일반 관련 드라이버 설치 파일 추가

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

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

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 22, 2024

@hakueon
타깃 브랜치가 master로 되어 있습니다.
develop-hw로 수정 바랍니다.

@hakueon hakueon changed the base branch from master to develop-hw October 22, 2024 07:09
@hakueon
Copy link
Contributor Author

hakueon commented Oct 22, 2024

변경하였습니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 23, 2024

@hakueon
추가 문제가 발견되어 공유드립니다.
기존의 드라이버 파일을 수정한 내용이 확인되었습니다, 'PROBO_Easy_Down_Entry_ver8.38_install.exe' → 'PROBO Easy Down Entry ver8.41.msi'

이 파일은 타사인 '프로보커넥트'에서 사용하는 드라이버 입니다.
파일명을 수정하거나 내용을 변경하면 해당 회사의 기기 연결을 불가능해집니다.
타사의 파일은 절대 수정해선 안됩니다.
유의 부탁드립니다.

@hakueon
Copy link
Contributor Author

hakueon commented Oct 23, 2024

@Tnks2U
안녕하세요!
미리 설명을 드렸어야 했는데 죄송합니다.
'프로보커넥트' 제품을 사용하고 있는 회사가 당사인 '에이럭스'와 합병되어 '프로보커넥트' 제품을 같이 판매 및 관리하고 있습니다.(인적, 물적 자원 모두 한 회사로 통합 됨)
코드에 문제가 없다면 병합 요청 드립니다.
감사합니다.

@Tnks2U
Copy link
Contributor

Tnks2U commented Oct 24, 2024

@hakueon
그렇군요.
PR마감기한이어서 우선 merge하겠습니다.
다만, 말씀하신 내용을 증명할 수 있는 서류 등을 메일로 제공받을 수 있을까요?

@Tnks2U Tnks2U merged commit b3fe00a into entrylabs:develop-hw Oct 24, 2024
2 checks passed
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