osc.js icon indicating copy to clipboard operation
osc.js copied to clipboard

Closes #204 - Add WebSerial support.

Open patrykwegrzyn opened this issue 2 years ago • 5 comments

Hi @colinbdclark i'm not familiar with testing framework you are using so this may need some work on your end. Please review and let me know if is anything else i can do to help. Best regards Pat

patrykwegrzyn avatar Apr 19 '23 21:04 patrykwegrzyn

Hi @patrykwegrzyn, thanks so much for this PR. I'm swamped at the moment but will definitely take a close look and review this as soon as I can.

It looks like your editor maybe has automatically reformatted a number of files that you probably didn't mean to change. Can you make sure any white space changes etc. are omitted from your PR so that it's easier to review what has actually changed? Thanks so much!

colinbdclark avatar Apr 20 '23 16:04 colinbdclark

Hi @colinbdclark can you let me know what settings you are using ? im using prettier with standard settings i may be able to and do it but it may be painful. As alternative I can make comments for you to highlight to what i have changed, i only added 1 line to Gruntfile , modified browser test and added osc-web-serialport.js . I did run build tho , probably that's why it looks like a lot stuff was changed.

patrykwegrzyn avatar Apr 20 '23 19:04 patrykwegrzyn

Hi, just checking if this PR will ever get accepted? I am getting a lot of Webpack 5 issues due to polyfill not being supported anymore. I can revert but, would love to upgrade my project

ahfontaine avatar Jul 19 '23 21:07 ahfontaine

Hi @ahfontaine, I am in the midst of a big career transition that has been occupying my time recently, but the good news is that I anticipate having more time to focus on my open source projects soon. I'm sorry for the delay with this, and do have the intention of reviewing and merging when life permits.

colinbdclark avatar Jul 31 '23 16:07 colinbdclark

@colinbdclark Any updates on this?

coder0107git avatar Dec 15 '23 02:12 coder0107git