refactor: 🎨 cleaned a few things up

examples/cli/.perseus/src/lib.rs

@@ -46,8 +46,6 @@ pub fn run() -> Result<(), JsValue> {
         templates => &get_templates_map(),
         locales => &get_locales()
     }
-    // TODO integrate templates fully
-    // BUG router sees empty template and moves on, fixed by above
 
     sycamore::render_to(
         || {
@@ -72,7 +70,6 @@ pub fn run() -> Result<(), JsValue> {
                         // If the user is using i18n, then they'll want to detect the locale on any paths missing a locale
                         // Those all go to the same system that redirects to the appropriate locale
                         // Note that `container` doesn't exist for this scenario
-                        // TODO redirect doesn't work until reload
                         RouteVerdict::LocaleDetection(path) => detect_locale(path.clone(), get_locales()),
                         // We handle the 404 for the user for convenience
                         // To get a translator here, we'd have to go async and dangerously check the URL

examples/showcase/src/lib.rs

@@ -10,7 +10,6 @@ define_app! {
         crate::templates::index::get_template::<G>(),
         crate::templates::about::get_template::<G>(),
         crate::templates::new_post::get_template::<G>(),
-        // BUG: Sycamore doesn't support dynamic paths before dynamic segments (https://github.com/sycamore-rs/sycamore/issues/228)
         crate::templates::post::get_template::<G>(),
         crate::templates::ip::get_template::<G>(),
         crate::templates::time_root::get_template::<G>(),

packages/perseus/src/shell.rs

@@ -7,7 +7,6 @@ use crate::ErrorPages;
 use std::cell::RefCell;
 use std::collections::HashMap;
 use std::rc::Rc;
-use sycamore::prelude::Template as SycamoreTemplate;
 use sycamore::prelude::*;
 use wasm_bindgen::prelude::*;
 use wasm_bindgen::JsCast;