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

Can not load iooxa.min.js #16

Closed
J2thearo opened this issue Jul 7, 2020 · 6 comments
Closed

Can not load iooxa.min.js #16

J2thearo opened this issue Jul 7, 2020 · 6 comments
Assignees
Labels

Comments

@J2thearo
Copy link

J2thearo commented Jul 7, 2020

Hello,
I'm very curious about iooxa. I was going to replace ink-components with it today, but I got a bug. Here is the basic index.html

Screenshot 2020-07-07 21 10 38

I keep getting the error

Screenshot 2020-07-07 21 13 04

I have also already downloaded iooxa.min.js locally but still get the error.

It would be great if you could help me out, thanks!

Many Greetings,
J2thearo

@rowanc1
Copy link
Member

rowanc1 commented Jul 7, 2020

Hm, interesting - there could be a race condition with adding the web-components. I will investigate this in the next few days and get back to you here!

What browser/OS are you using?

@rowanc1 rowanc1 added the bug label Jul 7, 2020
@rowanc1 rowanc1 self-assigned this Jul 7, 2020
@rowanc1
Copy link
Member

rowanc1 commented Jul 10, 2020

Hey @J2thearo,

Starting to look at this today. I think I know where the bug is and will fix it.

I think this shouldn't actually stop the article from loading and becoming interactive though?

Can you try copying this locally?
https://jsbin.com/qulahusifa/5/edit?html,output

I do get that error, but everything actually works fine!

rowanc1 added a commit that referenced this issue Jul 10, 2020
@rowanc1
Copy link
Member

rowanc1 commented Jul 10, 2020

This should now be fixed in the deployed version:

Note that you may have to explicitly point your script at it, the forwarding sometimes takes a while to update.

<script async src="https://unpkg.com/@iooxa/article@0.2.7/dist/iooxa.min.js"></script>

@rowanc1
Copy link
Member

rowanc1 commented Jul 10, 2020

Please let us know if you have any other problems upgrading from ink!!!

@rowanc1 rowanc1 closed this as completed Jul 10, 2020
@J2thearo
Copy link
Author

Hello Rowan,
thanks for taking care of it. Yeah, you're right, I can use iooxa normally despite the bug. In my original code, I messed up ink and iooxa syntax, so I couldn't get started. I was sure it was because of the bug, but in the end I just have to get used to the new syntax. At the moment I'm going deeper into the documentation and I'm sure there is still a lot to learn.

@rowanc1
Copy link
Member

rowanc1 commented Jul 13, 2020

Also, the best place to look for upgrading is here:
https://github.com/iooxa/article/releases/tag/v0.2.4

The name/value/bind was the pieces that likely give the most trouble. If you come across other things that don't make sense, don't hesitate to call them out as bugs! At the very least it means updating the documentation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants