Skip to content

Conversation

@Tnks2U
Copy link
Collaborator

@Tnks2U Tnks2U commented Apr 4, 2023

No description provided.

ImgBotApp and others added 30 commits February 1, 2023 16:58
*Total -- 970.05kb -> 157.61kb (83.75%)

/images/block_icon/hardware_bzr2.svg -- 247.28kb -> 0.91kb (99.63%)
/images/block_icon/hardware_dc.svg -- 248.25kb -> 1.61kb (99.35%)
/images/block_icon/hardware_led.svg -- 249.30kb -> 1.90kb (99.24%)
/images/hw/iCOBOT_Front.png -- 174.26kb -> 105.80kb (39.28%)
/images/beta_tag.svg -- 0.43kb -> 0.34kb (20.67%)
/images/hardware/KKMOO.png -- 50.53kb -> 47.05kb (6.89%)

Signed-off-by: ImgBotApp <[email protected]>
[#5764] 신규 판단, 값블록 구현
[5760] 하드웨어 최소버전안내문구 entrylms => modal로 변경
- 다국어, 블록메뉴 위치 수정
[#5764] 신규 판단, 값블록 구현 추가 수정
[4.23.0] 하드웨어 1.9.40 PR
@Tnks2U Tnks2U merged commit 823fb54 into develop-hw Apr 4, 2023
try {
await Promise.all(this.execPromises);
} catch (e) {}
} catch (e) { }
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Delete ·

Suggested change
} catch (e) { }
} catch (e) {}

resolve(null);
}
);
Entry.modal.alert(content, title, {
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Insert ⏎····················

Suggested change
Entry.modal.alert(content, title, {
Entry.moda
l.alert(content, title, {

}
);
Entry.modal.alert(content, title, {
withDontShowAgain: true
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Replace withDontShowAgain:·true with ····withDontShowAgain:·true,

Suggested change
withDontShowAgain: true
withDontShowAgain: true,e

Comment on lines +638 to +639
withDontShowAgain: true
}).then((data: { dontShowChecked: boolean }) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Replace ················}) with ····················})⏎····················

Suggested change
withDontShowAgain: true
}).then((data: { dontShowChecked: boolean }) => {
withDontShowAgain: true })
).then((data: { dontShowChecked: boolean }) => {

Entry.modal.alert(content, title, {
withDontShowAgain: true
}).then((data: { dontShowChecked: boolean }) => {
const { dontShowChecked } = data || {};
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Insert ····

Suggested change
const { dontShowChecked } = data || {};
const { dontShowChecked } = data || {};

const { dontShowChecked } = data || {};
if (dontShowChecked) {
localStorage.setItem('skipNoticeHWOldVersion', 'true');
}
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Insert ····

Suggested change
}
}

localStorage.setItem('skipNoticeHWOldVersion', 'true');
}
resolve(null);
})
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Replace }) with ····});

Suggested change
})
});)

localStorage.setItem('skipNoticeHWOldVersion', 'true');
}
resolve(null);
})
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] <semi> reported by reviewdog 🐶
Missing semicolon.

Suggested change
})
};)

contentResult += `${this.Block(block)}\n`;
try {
if (index === 0 && Entry.TextCodingUtil.isEventBlock(block)) {
rootResult = `${this.Block(block)}\n`;
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] <new-cap> reported by reviewdog 🐶
A function with a name starting with an uppercase letter should only be used as a constructor.

if (index === 0 && Entry.TextCodingUtil.isEventBlock(block)) {
rootResult = `${this.Block(block)}\n`;
} else {
contentResult += `${this.Block(block)}\n`;
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] <new-cap> reported by reviewdog 🐶
A function with a name starting with an uppercase letter should only be used as a constructor.

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