Skip to content
This repository has been archived by the owner on Jan 1, 2023. It is now read-only.

Websockets don't work with Firefox? #8

Closed
isaacabraham opened this issue Oct 13, 2018 · 7 comments · Fixed by #10
Closed

Websockets don't work with Firefox? #8

isaacabraham opened this issue Oct 13, 2018 · 7 comments · Fixed by #10

Comments

@isaacabraham
Copy link
Contributor

On a clean build from the latest version of the repository, whilst I can get "live changes" to occur when publishing posts in Chrome, it doesn't work in Firefox. Also - is it supposed to work for other actions than publishing e.g. update, delete etc.?

@Nhowka
Copy link

Nhowka commented Oct 13, 2018

If I remember correctly it was removed when migrating to the beta version of fable 2 because bridge was not compatible. I released a version recently, so it could be added again.

@Nhowka
Copy link

Nhowka commented Oct 13, 2018

Looks like the migration isn't merged yet, so it's probably not it... I will test it too to find what is causing it

@Zaid-Ajaj
Copy link
Owner

Thanks @Nhowka for taking a look, I will finish the migration to Fable 2 stable soon, then I will be able to use latest Elmish.Bridge and see if the issue persists

is it supposed to work for other actions than publishing e.g. update, delete etc.?

@isaacabraham For now only when publishing, other actions can be easily added but I started with this as proof-of-concept

@Zaid-Ajaj
Copy link
Owner

Zaid-Ajaj commented Oct 13, 2018

It is probably this issue and that I need to update Suave, @Nhowka Maybe you can publish a new version of Elmish.Bridge.Suave that requires Suave 2.5.2?

Edit: scratch that, I am not using latest

@isaacabraham
Copy link
Contributor Author

Thanks for the help both of you. Regarding publishing - I thought that was the case, but just wanted to make sure - I'm giving a demo on F# next week and wanted to show this as well :-)

@isaacabraham
Copy link
Contributor Author

@Zaid-Ajaj confirmed just pulled latest and all works now. Cheers.

@Zaid-Ajaj
Copy link
Owner

Glad to hear that! thanks for taking a look 😄

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

Successfully merging a pull request may close this issue.

3 participants