refactor #29

Merged
flaxeneel2 merged 8 commits from refactor into master 2026-06-20 22:43:46 +02:00
Owner

This PR broke down some of the existing large files into smaller ones. This is another PR i wanted to push through before i start the experimentation of slint stack ( Issue #27 ) .

This PR broke down some of the existing large files into smaller ones. This is another PR i wanted to push through before i start the experimentation of slint stack ( Issue #27 ) .
human489 approved these changes 2026-06-20 20:01:44 +02:00
Dismissed
@ -25,2 +8,4 @@
mod password_auth;
mod registration;
pub struct ClientHandler {
Owner

We should probably rename this and the entire file actually to something that's a bit more accurate I suppose, but that's a minor nitpick, because I like this file being a kind of index file!

We should probably rename this and the entire file actually to something that's a bit more accurate I suppose, but that's a minor nitpick, because I like this file being a kind of index file!
@ -0,0 +17,4 @@
pub use account::reset_account;
pub use auth::{login, logout, oauth_login, oauth_register, register, restore_session};
pub use dm::get_dm_rooms;
#[allow(deprecated)]
Owner

👀
tick tock...

👀 tick tock...
@ -91,3 +75,1 @@
};
Ok(Some((stronghold, client.store(), key_provider, snapshot_path)))
let opened = open_store(&key_provider, &snapshot_path, user_id, create_if_missing)?;
Owner

bless your heart

bless your heart
rayyan marked this conversation as resolved
Owner

Everything looks fine, goodbye to user.rs finally...
Just two minor nitpicks above, other than that we should be good to merge and start work on frontend refactor after I make sure actions are fine :D

Everything looks fine, goodbye to user.rs finally... Just two minor nitpicks above, other than that we should be good to merge and start work on frontend refactor after I make sure actions are fine :D
Merge branch 'master' into refactor
All checks were successful
build-test / test-tauri (, ubuntu-22.04) (push) Successful in 3m16s
build-test / test-tauri (, ubuntu-22.04) (pull_request) Successful in 6m59s
321a1ff158
flaxeneel2 dismissed human489's review 2026-06-20 22:20:08 +02:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

flaxeneel2 scheduled this pull request to auto merge when all checks succeed 2026-06-20 22:42:41 +02:00
rayyan approved these changes 2026-06-20 22:43:45 +02:00
rayyan left a comment

We can resolve nitpicks later, go ahead and merge

We can resolve nitpicks later, go ahead and merge
flaxeneel2 deleted branch refactor 2026-06-20 22:44:07 +02:00
Sign in to join this conversation.
No description provided.