Protected Route condition

[MFJoyBoy]

ProtectedRoute and ProtectedParentRoute require condition field of type Option
it's like resource, so i thought maybe resource could be a good option here but if i create a resource and pass it right away (in the closure) it will get a lot of errors in console (it keep trying to make a call to server in endless loop)
any comment about to pass a resource to condition field of the protected route would be appreciated (or if it's good idea in first place)

[gbj]

Yes, it is designed to work with resources and it should work with resources. It will not create an infinite loop if it is done correctly. Are you creating the resource inside the condition check? Please provide some example code, otherwise it's not possible to help.

[MFJoyBoy]

this is the code i tried:

     condition: || {
            let user = OnceResource::new(async {
                let response = get_current_user_private_data().await;
                match response {
                    Ok(value) => value.admin(),
                    Err(error) => {
                        log!("{error:?}");
                        false
                    }
                }
            });
            user.get()
        },

[MFJoyBoy]

hmm i feel like i should put resource definition somewhere else but not sure where

[gbj]

Right, in this example you are creating the resource inside the same condition check which reactively reads from the resource, which means that you are creating an infinite loop.

[MFJoyBoy]

i feel silly with the problems i counter with leptos 😆
well that could be the reason why people avoid using system language for high level programming, things could get tricky easily compare to js framework like angular
(i am absolutely okay with it, love rust and enjoyed leptos so far)

[MFJoyBoy]

this seems to work

pub fn panel_routes() -> impl MatchNestedRoutes + Clone {
    let user = OnceResource::new(async {
        let response = get_current_user_private_data().await;
        match response {
            Ok(value) => value.admin(),
            Err(error) => {
                log!("{error:?}");
                false
            }
        }
    });

    ProtectedParentRoute(ProtectedParentRouteProps {
         condition: move || user.get(),
         ...
    

[gbj]

Yes, this is the right way to do it.