Skip to content

Conversation

@Tnks2U
Copy link
Contributor

@Tnks2U Tnks2U commented May 22, 2023

하드웨어 1.9.42

신규 하드웨어


기존 하드웨어

  • [표쌤코딩] 공유보드 (공유보드 업데이트 2305 entryjs#2496, Develop hw #630) (샘플없음)
    • ''터치 %1 센서값'' 블록 제거
    • 펌웨어, 썸네일 이미지 업데이트
  • [네오피아] NEO, 뉴 네오봇, 네오쏘코, 네오씽카 동시 수정([네오피아] 버그 수정 entryjs#2503) (샘플있음)
    • 연결 안전성 업데이트
    • lcd 블럭의 옵션값이 숫자로 나오던 문제를 이미지이름으로 나오도록 수정
    • 블럭 카테고리 구분을 위한 타이틀 추가
    • 모터 왼쪽 오른쪽이 뒤바뀌던 버그 수정
    • 웹연결에도 동일하게 적용

dalgona-edu and others added 30 commits April 6, 2023 17:21
잘못된 적용삭제
잘못된 파일
잘못된 파일
하드웨어 이미지 교체/블록 추가
하드웨어 이미지 교체/블록 추가
하드웨어 이미지 변경, 블록 추가 및 수정
This reverts commit 71a2b1c.
This reverts commit 87f36f7.
This reverts commit f2bd8d2.
This reverts commit d756487.
@Tnks2U Tnks2U requested review from kimorkim, lizwan and prisml May 22, 2023 09:15
@Tnks2U Tnks2U merged commit ed9097e into master May 22, 2023
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

app/modules/dalgona.js|1051 col 34| Unexpected space before function parentheses.
app/modules/dalgona.js|1058 col 43| Unexpected space before function parentheses.
app/modules/neo_cannon.js|194 col 13| Expected { after 'if' condition.


let sensorIdx = 0;

Module.prototype.init = function (handler, config) {};

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.init = function (handler, config) {};
Module.prototype.init = function(handler, config) {};


Module.prototype.init = function (handler, config) {};

Module.prototype.setSerialPort = function (sp) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.setSerialPort = function (sp) {
Module.prototype.setSerialPort = function(sp) {

this.sp = sp;
};

Module.prototype.requestInitialData = function () {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.requestInitialData = function () {
Module.prototype.requestInitialData = function() {

//return this.makeSensorReadBuffer(this.sensorTypes.ANALOG, 0);
};

Module.prototype.checkInitialData = function (data, config) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.checkInitialData = function (data, config) {
Module.prototype.checkInitialData = function(data, config) {

// return isValidData;
};

Module.prototype.afterConnect = function (that, cb) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.afterConnect = function (that, cb) {
Module.prototype.afterConnect = function(that, cb) {

if ($.isPlainObject(data)) {
textLen = ('' + data.text).length;
// console.log(textLen);
text = Buffer.from('' + data.text, 'ascii');

Choose a reason for hiding this comment

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

⚠️ [eslint] <prefer-template> reported by reviewdog 🐶
Unexpected string concatenation.

Suggested change
text = Buffer.from('' + data.text, 'ascii');
text = Buffer.from(`${ data.text}`, 'ascii');

case this.sensorTypes.MP3PLAY2: {
const tx = new Buffer(2);
const num = new Buffer(2);
const time_value = new Buffer(2);

Choose a reason for hiding this comment

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

⚠️ [eslint] <camelcase> reported by reviewdog 🐶
Identifier 'time_value' is not in camel case.


buffer = new Buffer([255, 85, 10, sensorIdx, this.actionTypes.SET, device, port]);

buffer = Buffer.concat([buffer, tx, num, time_value, dummy]);

Choose a reason for hiding this comment

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

⚠️ [eslint] <camelcase> reported by reviewdog 🐶
Identifier 'time_value' is not in camel case.

return buffer;
};

Module.prototype.getDataByBuffer = function (buffer) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.getDataByBuffer = function (buffer) {
Module.prototype.getDataByBuffer = function(buffer) {

return datas;
};

Module.prototype.disconnect = function (connect) {

Choose a reason for hiding this comment

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

⚠️ [eslint] <space-before-function-paren> reported by reviewdog 🐶
Unexpected space before function parentheses.

Suggested change
Module.prototype.disconnect = function (connect) {
Module.prototype.disconnect = function(connect) {

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.

5 participants