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

Move pages used by a normal user to svelte #570

Merged
merged 2 commits into from May 9, 2023
Merged

Conversation

PeterJFB
Copy link
Contributor

@PeterJFB PeterJFB commented Apr 17, 2023

Work in progress Finished ™️

What has been added?

  • The page to vote ☑️
  • The page to verify your vote /retrieve
  • An alert-service called alerts (should behave in the same way as the previous, with some changes to the api)
  • socketIOService and the use of socket-io in development mode
  • A logout button :o

Additional changes

  • Rewrite css to scss, allowing more expressive styling
  • Simplified routing logic some more
  • Retrieved backend types with the $backend alias

TODO

  • Testing ... a lot of testing ...
  • Maybe also write tests
  • Still have some bugged behavior with sortablejs, consider looking some more for a usable library

message,
type,
fade,
id: Math.round(Math.random() * 10000),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. Use UUID or something other that's actually predictably non-clashing.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yolo

Comment on lines +17 to +25
// https://github.com/axe-me/vite-plugin-node/blob/main/packages/vite-plugin-node/src/server/express.ts
// production's io-server is in server.ts
adapter({ app, server, req, res }) {
if (env.NODE_ENV !== 'production' && !app.get('io')) {
const io = new Server(server.httpServer);
app.set('io', io);
}
app(req, res);
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome 😍

Copy link

@Arashfa0301 Arashfa0301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PeterJFB PeterJFB merged commit c947734 into master May 9, 2023
3 checks passed
@PeterJFB PeterJFB deleted the svelte-normaluser branch May 9, 2023 17:00
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.

None yet

3 participants