Skip to content

Nuxt rebuild (3rd try)#341

Closed
Tobi406 wants to merge 27 commits into
LuckPerms:nuxt-rebuildfrom
Tobi406:nuxt-rebuild-3
Closed

Nuxt rebuild (3rd try)#341
Tobi406 wants to merge 27 commits into
LuckPerms:nuxt-rebuildfrom
Tobi406:nuxt-rebuild-3

Conversation

@Tobi406
Copy link
Copy Markdown
Contributor

@Tobi406 Tobi406 commented Dec 3, 2020

All good things are threes, right? Atleast I hope, else I'll soon reach double-digits with my attempts to integrate the website with Nuxt.

What is already working?

  • Verbose
  • Treeview
  • "normal" pages (home, sponsor, download)

What is not yet working?

  • Editor
    • the vue-intersect package has the issue that it uses HTMLElement as a type which isn't recognized by the server (see on GitHub)
    • the communicating with the store doesn't seem to work and no sessions can be opened
  • Wiki
    • the main issue is that on the first page switch it shows a 404 and it crashes after too many or too fast page switches
    • page load times are a little bit increased, but that should be acceptable to the end-user (even tough I hope we can improve them)
  • Meta images
    • The image on the Discord embed doesn't show, probably just something with me setting the meta tag content wrong, should be easily fixable
  • The hash on the footer

Todo (after everything is working)

  • Make sure all the code is up to date
  • Remove potential Vue things we don't need for Nuxt

Demo

As always: https://tobi406-luckpermsweb-dev.netlify.app/

@Tobi406
Copy link
Copy Markdown
Contributor Author

Tobi406 commented Dec 4, 2020

Ok, so, I've run into the same issue again as in #283 😭 We can pregenerate /editor and /editor/demo, but not any dynamic ids, In an nuxt issue a recommended solution was to use query params, which we have already discussed on the 2nd attempt.

@Tobi406 Tobi406 closed this Dec 5, 2020
@BrainStone
Copy link
Copy Markdown
Contributor

The original editor loaded the data with JavaScript clientside. What’s wrong with that technique here?

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.

2 participants