osc.js-examples icon indicating copy to clipboard operation
osc.js-examples copied to clipboard
verified publisher icon colinbdclark

Unsupported relay() methods in examples?

Open davydka opened this issue 4 years ago • 1 comments

Looking through the examples, it seems that .Relay() is used. When debugging and building my app, I went to troubleshoot the Relay() method and the source code indicates that this is an Unsupported, non-API function..

My question/issue is: Should we be presenting unsupported functions in the official examples for OSC.js? If not, we should update this example.

Relay() in examples: https://github.com/colinbdclark/osc.js-examples/blob/b1bc1aa34bb15a5f06ce24a8a46f9c7e7c62cfd8/udp-browser/index.js#L55

Relay() in source code: https://github.com/colinbdclark/osc.js/blob/3bfa3d4489497b04a806298b1f502ffbbdfed2df/src/osc-transports.js#L133

davydka avatar Oct 02 '21 00:10 davydka

Unsupported, non-API functions probably shouldn't be in the official examples, no. I'll happily take a PR that either provides a reasonable replacement for it in the examples, or a PR to the osc.js repository that addresses the issues in osc.Relay's implementation.

colinbdclark avatar Mar 14 '23 22:03 colinbdclark