Skip to content
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

examples が browser/node で分かれているのを纏められないか #101

Open
dynamis opened this issue Jun 10, 2021 · 6 comments
Open
Labels
enhancement New feature or request

Comments

@dynamis
Copy link
Contributor

dynamis commented Jun 10, 2021

browser/node 版それぞれを使う人にとっては必要十分な情報がこちらにある

但し、少し気になるところがある

また、Remote 版のサーバは node 版も用意すれば自由に組み合わせて使えるようになったりするし、browser/node で分けずに CHIRIMEN のデバイスの説明とそこからそれぞれの利用例をリンクするような形にした方が良さそうに思う。

@kou029w
Copy link
Contributor

kou029w commented Jun 10, 2021

https://github.com/chirimen-oh/remote-connection/blob/master/js/RelayServer.js 読むと下記のAPIがNode.jsではデフォルトでglobalThisに存在しないと思うので対処が必要そうですね:

  • WebSocket
  • TextEncoder
  • Scaledrone

@dynamis
Copy link
Contributor Author

dynamis commented Jun 10, 2021

Scaledrone はサービス側がライブラリとして提供してるヤツで、こいつの node 向けライブラリがあるのかという問題になりますね
<script src='https://cdn.scaledrone.com/scaledrone-lite.min.js'></script>
https://github.com/chirimen-oh/remote-connection#scaledrone

@dynamis
Copy link
Contributor Author

dynamis commented Jun 10, 2021

TextEncoder って MDN 的には Node 11.0 からサポート的に書いてるけど使えないのかな。
使えないなら使えないでここに書いてある Polifill も一緒に突っ込んでも良いけど。
https://developer.mozilla.org/ja/docs/Web/API/TextEncoder

@kou029w
Copy link
Contributor

kou029w commented Jun 10, 2021

https://github.com/chirimen-oh/chirimen/blob/aa74b01e86c5a58acf14739f5980f84b8deac0eb/setup.sh#L95-L97

今 setup.sh 見ると v12 なので問題無さそうですね 🙆‍♂️ > TextEncoder

@dynamis
Copy link
Contributor Author

dynamis commented Jun 10, 2021

WebSocket はライブラリに入れ替えるか universal なモジュールに差し替える程度にするか?
https://www.npmjs.com/package/universal-websocket-client

どうやっても良いけど確かにちょっと手当は必要

@dynamis
Copy link
Contributor Author

dynamis commented Jun 10, 2021

Scaledrone はサービス側がライブラリとして提供してるヤツで、こいつの node 向けライブラリがあるのかという問題になりますね

オフィシャルドキュメント https://www.scaledrone.com/docs/api-clients/javascript は browser 向けのものしか案内してくれてないけど node module これ使えば良い感じかな。こいつが両対応で動けば分けずに済むが...
https://www.npmjs.com/package/scaledrone-node

ついでに scaledrone のモジュールは使う時に動的・自動的に読み込むようにして、利用するサービスに応じて script タグを追加しなくて良いように出来ると嬉しい

@dynamis dynamis added the enhancement New feature or request label Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants