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

point to @webxdc/types and use @webxdc/vite-plugins for emulator #61

Closed
wants to merge 3 commits into from

Conversation

adbenitez
Copy link
Collaborator

@adbenitez adbenitez commented Nov 27, 2024

close #51

@adbenitez adbenitez force-pushed the adb/point-to-webxdc-types-and-emulator branch from d7d3831 to e4f6c48 Compare November 27, 2024 08:38
Copy link
Contributor

@Simon-Laux Simon-Laux left a comment

Choose a reason for hiding this comment

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

Looks good to me, but let's wait for @r10s' approval

Copy link
Collaborator

@r10s r10s left a comment

Choose a reason for hiding this comment

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

first of all, thanks a lot for the cleanup of the other repo!

for webxdc.js on some CDN:

i understand, why we do not want to copy the file, wrt maintenance etc.

however, by pulling a file from some CDN, the magic of simplicity and understanding things when not deep into js is also gone somehow, at least for ppl as me.

when following the instructions, you will also have https://cdn.jsdelivr.net/gh/webxdc/vite-plugins@latest/src/webxdc.js forever in your source code ...

and these instructions are also not what we're saying at https://webxdc.org/docs/#a-simple-example

and having two webxdc.js in the code is also a bit weird.

so, regarding the fact that webxdc.js is not changed that often, maybe it is better to stay with a copy.

Comment on lines +7 to 8
<script src="https://cdn.jsdelivr.net/gh/webxdc/vite-plugins@latest/src/webxdc.js"></script>
<script src="webxdc.js"></script>
Copy link
Collaborator

Choose a reason for hiding this comment

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

so, that results to one 404 in all cases?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

one of them will always succeed, if you are in DC the real thing will load while in the browser the emulator is used

@adbenitez
Copy link
Collaborator Author

when following the instructions, you will also have https://cdn.jsdelivr.net/gh/webxdc/vite-plugins@latest/src/webxdc.js forever in your source code ...

that is not a problem since it just silently fails to load inside DC where the app doesn't have internet

however, by pulling a file from some CDN, the magic of simplicity and understanding things when not deep into js is also gone somehow, at least for ppl as me.

at the same time now it is easier for people that don't have experience with git, they can just download the index.html file and just with that file alone or copying the content of the example index.html can get it to work

@adbenitez adbenitez closed this Nov 27, 2024
@link2xt
Copy link
Contributor

link2xt commented Nov 28, 2024

I have made follow-up PRs #63 (removing types and pointing to the source where they can be downloaded if needed) and #64 (updating webxdc.js once so it points to the correct source where webxdc.js is maintained).

@link2xt
Copy link
Contributor

link2xt commented Nov 28, 2024

There is also an unrelated release-webxdc.yml in this PR, could be a separate PR.

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.

Is this a sample project or a dev tool?
4 participants