phetsims / phet-office-add-in

MIT License
0 stars 0 forks source link

Office-mix app appears broken #5

Closed oliver-phet closed 5 years ago

oliver-phet commented 7 years ago

A user emailed phethelp that the office mix app appears to be broken. Amy and I reproduced on our machines (both Mac), but it sounds like @ariel-phet also reproduced on his Windows.

When I try to 'insert' the PhET add-in, I get a brief loading animation, but then nothing happens.

screen shot 2017-03-21 at 12 57 21 pm
mattpen commented 7 years ago

@oliver-phet - I think it should be fixed now. It looks like the script that automatically updates it has a bug.

oliver-phet commented 7 years ago

Maybe there is a delay for the fix to come through? I just tried again and am getting the same behavior.

mattpen commented 7 years ago

image

Did you try pressing reload? That fixed it for me.

oliver-phet commented 7 years ago

Yep, I reloaded, removed and re-added the add-in, and restarted PP.

screen shot 2017-03-21 at 1 25 47 pm

After I click Reload, I get a brief loading animation, then the screen remains blank.

mattpen commented 7 years ago

It may be subject to caching on the Microsoft side. It is a static file and not cached in varnish, but the test environment is still showing errors too: https://labsjs.blob.core.windows.net/sdk/LabsJS-1.0.4/labshost.html?lab=https://phet.colorado.edu/office-mix/store.html?PostMessageLabHost

If you add an arbitrary query parameter, it works normally: https://labsjs.blob.core.windows.net/sdk/LabsJS-1.0.4/labshost.html?lab=https://phet.colorado.edu/office-mix/store.html?PostMessageLabHost&1234

I would check again in 15 minutes?

mattpen commented 7 years ago

@oliver-phet - Did you have a chance to check on the Mac again today? The app is still working for me as of today in the labsjs links above and in Powerpoint for Windows.

oliver-phet commented 7 years ago

Working again on my Mac. I'll close for now.

oliver-phet commented 5 years ago

@mattpen This appears to be happening again. Can you see if the same solution you did before can fix?

mattpen commented 5 years ago

This is a new problem, the update script is working as expected.