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

WASM #1024

Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

WASM #1024

wants to merge 46 commits into from

Conversation

calledude
Copy link
Contributor

@calledude calledude commented Sep 1, 2024

This PR is most easily read by going through the commits one by one.

Due to responsiveness being quite sluggish when the user has high latency to the server, I rewrote the webpage into WASM

I tried getting InteractiveAuto to work, but we will have to introduce some sort of abstraction to the logic in each controller, so that the server can use those implementations instead of trying to go via a controller
I opted to not do this in this PR due to it being large/long-winded as it is.

I also had to move some models (enums etc) to a shared project, so that the wasm artifact didn't become massive and bring with it a bunch of unnecessary stuff

NOTE: The GuildOption dropdown is slow to load because it's currently bypassing the pre-rendering step due to the server not having BaseAddress set.

Move Routes/MainLayout component to Wasm project to enable Auto rendering mode
…internal authentication towards the AuthenticationService
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.

1 participant