LemmyNet
/
lemmy
3
2
Fork 3
Code Pull Requests 1 Releases Wiki Activity

Refactor websocket to split it into multiple files #91

Merged
dessalines merged 1 commits from refactor-websocket into main 2020-08-31 16:43:17 +00:00
Conversation 0 Commits 1 Files Changed 20 +470 -437
nutomic commented 2020-08-31 15:25:35 +00:00
Owner
Copy Link

No functional changes, just trying to simplify the code a bit. Websocket still works fine based on my quick testing.

No functional changes, just trying to simplify the code a bit. Websocket still works fine based on my quick testing.
dessalines reviewed 2020-08-31 15:35:34 +00:00
server/src/websocket/messages.rs
@ -0,0 +134,4 @@
pub struct CheckCaptcha {
pub uuid: String,
pub answer: String,
}
dessalines commented 2020-08-31 15:35:34 +00:00
Owner
Copy Link

I do like the splitting out of messages and handlers, makes sense. I'll test and then merge if its good.

I do like the splitting out of messages and handlers, makes sense. I'll test and then merge if its good.
dessalines merged commit c4dd28e252 into main 2020-08-31 16:43:17 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: LemmyNet/lemmy#91
No description provided.
Delete Branch "refactor-websocket"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?