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

Performance Enhancements, New Features, and Bug Fixes #37

Merged
merged 10 commits into from
Jun 26, 2024
Merged

Conversation

maxentr
Copy link
Owner

@maxentr maxentr commented Jun 26, 2024

Performance Improvements

  • Web: Reduced API calls when using sliders in the lobby
  • General: Implemented messagePack parser with socket.io for enhanced performance

New Features

  • Web: Added keyboard navigation support for the radio-number component

Bug Fixes

  • Web: Prevented users from tabbing into game chat when in the lobby

Refactoring

  • Web: Removed reconnection parameters to utilize default values

Build Enhancements

  • API: Added bufferutil and utf-8-validate to improve WebSocket speed
  • Updated and fixed pnpm-lock.yaml.

Chore

  • API: Enabled CORS for the API server
  • General: Updated lefthook.yml to check only staged files.

Copy link

vercel bot commented Jun 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
skyjo ⬜️ Ignored (Inspect) Visit Preview Jun 26, 2024 8:54pm

@maxentr maxentr changed the title Develop Performance Enhancements, New Features, and Bug Fixes Jun 26, 2024
Copy link

github-actions bot commented Jun 26, 2024

Coverage Report for API

Status Category Percentage Covered / Total
🟢 Lines 99.51% (🎯 90%) 411 / 413
🟢 Statements 99.56% (🎯 90%) 462 / 464
🟢 Functions 99.29% (🎯 90%) 140 / 141
🟢 Branches 99.43% (🎯 90%) 177 / 178
File CoverageNo changed files found.
Generated in workflow #27

@maxentr maxentr merged commit fe782a3 into main Jun 26, 2024
2 checks passed
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