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

178 handle all websocket messages from the backend #228

Merged
merged 4 commits into from
Dec 5, 2024

Conversation

SebastianNowak01
Copy link
Collaborator

This pull request includes several changes to both the backend and frontend components of the project, focusing on improving type safety, handling WebSocket messages, and refining form validation. The most important changes are grouped by theme below. Closes #178.

WebSocket Message Handling:

Type Safety Improvements:

Form Validation:

Enum Serialization:

@SebastianNowak01 SebastianNowak01 added the frontend Related to the frontend label Dec 4, 2024
@SebastianNowak01 SebastianNowak01 self-assigned this Dec 4, 2024
@SebastianNowak01 SebastianNowak01 linked an issue Dec 4, 2024 that may be closed by this pull request
Copy link

deepsource-io bot commented Dec 4, 2024

Here's the code health analysis summary for commits a050756..5453121. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource Rust LogoRust✅ SuccessView Check ↗
DeepSource JavaScript LogoJavaScript✅ Success
❗ 4 occurences introduced
🎯 3 occurences resolved
View Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

frontend/src/routes/report/InputRow.tsx Outdated Show resolved Hide resolved
frontend/src/routes/report/page.tsx Outdated Show resolved Hide resolved
frontend/src/routes/report/tables.tsx Outdated Show resolved Hide resolved
frontend/src/routes/report/tables.tsx Outdated Show resolved Hide resolved
frontend/src/routes/report/types.ts Outdated Show resolved Hide resolved
frontend/src/routes/report/websocketMethods.tsx Outdated Show resolved Hide resolved
@m-milek m-milek merged commit 989ed02 into dev Dec 5, 2024
15 checks passed
@m-milek m-milek deleted the 178-handle-all-websocket-messages-from-the-backend branch December 9, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Related to the frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Handle all WebSocket messages from the backend
2 participants