search

dotnet-websharper / mvu

Model-View-Update architecture for WebSharper client-side applications.
https://dotnet-websharper.github.io/mvu/
Apache License 2.0
25 stars 3 forks source link

Prompt to add local storage before RemoteDev #1

Closed teo-tsirpanis closed 6 years ago

teo-tsirpanis commented 6 years ago

If RemoteDev is added before local storage, when the page loads, it will consider the initial model passed in App.Create as the initial one, and only after the first update will it update the model to the actual one.

Tarmil commented 6 years ago

Actually, I think it should be possible to fix the library so that the ordering doesn't matter. That would be even better, don't you think? I haven't tried, but I think all it would take is to call the original app.Init() at the end instead of the beginning of the new init here.

teo-tsirpanis commented 6 years ago

Yes, it is. I just proposed a temporary solution. I don’t have time to fix it more deeply.

Sent from Mail for Windows 10

From: Loïc Denuzière Sent: Πέμπτη, 26 Απριλίου 2018 13:16 To: dotnet-websharper/mvu Cc: Theodore Tsirpanis; Author Subject: Re: [dotnet-websharper/mvu] Prompt to add local storage beforeRemoteDev (#1)

Actually, I think it should be possible to fix the library so that the ordering doesn't matter. That would be even better, don't you think? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

Tarmil commented 6 years ago

Ok I'll merge this in the meantime.