-
Notifications
You must be signed in to change notification settings - Fork 497
[4.21.0] 하드웨어 1.9.38 PR #2434
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
[4.21.0] 하드웨어 1.9.38 PR #2434
Conversation
Tnks2U
commented
Jan 20, 2023
- https://oss.navercorp.com/entry/entry2/issues/5698
develop-hw 최신화
Issue/microbit eventblock
feat: 웹연결 버튼 조건 변경
This reverts commit acb5c24.
- 버튼 중복입력을 방지하기 위해 쓰로틀링 추가
Change the icobot_get_discomfort_index_value
[#5643] 마이크로비트 버튼이벤트 추가
feat: 버튼이벤트 블록 위치 수정
develop-hw 최신화
| // INFO: 디바이스가 모바일이거나 일렉트론이면 1차적으로 제외 | ||
| if (userAgentString.includes('mobile') || userAgentString.includes('electron')) { | ||
| return false; | ||
| } else if (userAgentString.includes('whale') || userAgentString.includes('edge') || userAgentString.includes('chrome')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| // INFO: 디바이스가 모바일이거나 일렉트론이면 1차적으로 제외 | ||
| if (userAgentString.includes('mobile') || userAgentString.includes('electron')) { | ||
| return false; | ||
| } else if (userAgentString.includes('whale') || userAgentString.includes('edge') || userAgentString.includes('chrome')) { |
There was a problem hiding this comment.
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 userAgentString.includes('whale')·||·userAgentString.includes('edge')·||·userAgentString.includes('chrome') with ⏎············userAgentString.includes('whale')·||⏎············userAgentString.includes('edge')·||⏎············userAgentString.includes('chrome')⏎········
| } else if (userAgentString.includes('whale') || userAgentString.includes('edge') || userAgentString.includes('chrome')) { | |
| } else if ( | |
| userAgentString.includes('whale') || | |
| userAgentString.includes('edge') || | |
| userAgentString.includes('chrome') | |
| ) { |