From 75a8f2d6d4faead2217c53179d7d0c8620171210 Mon Sep 17 00:00:00 2001
From: Lucas Vas <lvas463@gmail.com>
Date: Wed, 27 Aug 2025 13:20:16 -0400
Subject: [PATCH] chore: cleanup warnings and unused code

---
 src/app.rs                                 |  2 +-
 src/common/form_definition.rs              | 35 -----------
 src/common/loaner_user.rs                  | 18 ------
 src/common/mod.rs                          |  4 +-
 src/components/checkboxes.rs               |  2 +-
 src/components/multi_entry.rs              |  6 +-
 src/lib.rs                                 |  2 -
 src/pages/home_page.rs                     |  1 -
 src/pages/loaner_kiosk/action_selection.rs |  8 ---
 src/pages/loaner_kiosk/loaner_kiosk.rs     | 39 ------------
 src/pages/loaner_kiosk/mod.rs              |  3 -
 src/pages/loaner_kiosk/user_entry.rs       | 72 ----------------------
 src/pages/mod.rs                           |  1 -
 13 files changed, 6 insertions(+), 187 deletions(-)
 delete mode 100644 src/common/form_definition.rs
 delete mode 100644 src/common/loaner_user.rs
 delete mode 100644 src/pages/loaner_kiosk/action_selection.rs
 delete mode 100644 src/pages/loaner_kiosk/loaner_kiosk.rs
 delete mode 100644 src/pages/loaner_kiosk/mod.rs
 delete mode 100644 src/pages/loaner_kiosk/user_entry.rs

diff --git a/src/app.rs b/src/app.rs
index fe56c4b..ba349be 100644
--- a/src/app.rs
+++ b/src/app.rs
@@ -1,7 +1,7 @@
 use leptos::prelude::*;
 use leptos_meta::{provide_meta_context, MetaTags, Stylesheet, Title};
 use leptos_oidc::{
-    Auth, AuthLoaded, AuthParameters, AuthSignal, Authenticated, Challenge, LoginLink, LogoutLink,
+    Auth, AuthParameters, AuthSignal, Challenge, LoginLink,
 };
 use leptos_router::{
     components::{Route, Router, Routes},
diff --git a/src/common/form_definition.rs b/src/common/form_definition.rs
deleted file mode 100644
index 33fab4c..0000000
--- a/src/common/form_definition.rs
+++ /dev/null
@@ -1,35 +0,0 @@
-use serde::{Deserialize, Serialize};
-use std::collections::HashMap;
-
-#[derive(Serialize, Deserialize)]
-struct FormDefinition {
-    sections: Vec<Section>,
-}
-
-#[derive(Serialize, Deserialize)]
-struct Section {
-    questions: Vec<QuestionType>,
-}
-
-#[derive(Serialize, Deserialize)]
-enum QuestionType {
-    Text,
-    Email,
-    Number,
-    Dropdown,
-    Checkbox,
-    Radio,
-    Multi,
-}
-
-struct TextInput {}
-
-#[derive(Serialize, Deserialize)]
-struct QuestionComponent {
-    question_type: QuestionType,
-    label: Option<String>,
-    placeholder: Option<String>,
-
-    // Used in <select> and <checkbox> elements
-    options: Option<HashMap<String, String>>,
-}
diff --git a/src/common/loaner_user.rs b/src/common/loaner_user.rs
deleted file mode 100644
index a917743..0000000
--- a/src/common/loaner_user.rs
+++ /dev/null
@@ -1,18 +0,0 @@
-#[derive(Clone)]
-pub struct LoanerUser {
-    pub first_name: String,
-    pub last_name: String,
-    pub email: String,
-    pub has_loan: bool,
-}
-
-impl LoanerUser {
-    pub fn new() -> Self {
-        LoanerUser {
-            first_name: "".to_string(),
-            last_name: "".to_string(),
-            email: "".to_string(),
-            has_loan: false,
-        }
-    }
-}
diff --git a/src/common/mod.rs b/src/common/mod.rs
index 6deae78..33ed212 100644
--- a/src/common/mod.rs
+++ b/src/common/mod.rs
@@ -1,6 +1,4 @@
-mod form_definition;
-mod loaner_user;
 mod student_info;
 mod user;
 
-pub use self::{loaner_user::*, student_info::*, user::*};
+pub use self::{student_info::*, user::*};
diff --git a/src/components/checkboxes.rs b/src/components/checkboxes.rs
index 19f8b0f..cf894ca 100644
--- a/src/components/checkboxes.rs
+++ b/src/components/checkboxes.rs
@@ -1,4 +1,4 @@
-use leptos::{leptos_dom::logging::console_log, prelude::*};
+use leptos::prelude::*;
 
 #[component]
 pub fn Checkbox(
diff --git a/src/components/multi_entry.rs b/src/components/multi_entry.rs
index 0a9a5ce..c0e2725 100644
--- a/src/components/multi_entry.rs
+++ b/src/components/multi_entry.rs
@@ -1,7 +1,7 @@
 use std::collections::HashMap;
 
 use crate::components::ActionButton;
-use leptos::{leptos_dom::logging::console_log, prelude::*};
+use leptos::prelude::*;
 use serde::{Deserialize, Serialize};
 use uuid::Uuid;
 
@@ -32,7 +32,7 @@ pub struct MultiEntryMember {
 
 impl MultiEntryMember {
     /// Creates a new MultiEntryMember struct.
-    pub fn new() -> Self {
+    pub fn _new() -> Self {
         Self {
             display_name: String::new(),
             member_name: String::new(),
@@ -40,7 +40,7 @@ impl MultiEntryMember {
     }
 
     /// Creates a new MultiEntryMember struct with the specified information.
-    pub fn from(display_name: String, member_name: String) -> Self {
+    pub fn _new_with_data(display_name: String, member_name: String) -> Self {
         Self {
             display_name,
             member_name,
diff --git a/src/lib.rs b/src/lib.rs
index e1cbf90..9958e9f 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -1,5 +1,3 @@
-use leptos::prelude::event_target_value;
-
 pub mod app;
 mod common;
 mod components;
diff --git a/src/pages/home_page.rs b/src/pages/home_page.rs
index c28b9bf..27c3a80 100644
--- a/src/pages/home_page.rs
+++ b/src/pages/home_page.rs
@@ -1,4 +1,3 @@
-use std::collections::HashMap;
 #[cfg(feature = "ssr")]
 use aws_sdk_dynamodb::{error::ProvideErrorMetadata, types::AttributeValue, Client};
 use crate::app::Unauthenticated;
diff --git a/src/pages/loaner_kiosk/action_selection.rs b/src/pages/loaner_kiosk/action_selection.rs
deleted file mode 100644
index e410575..0000000
--- a/src/pages/loaner_kiosk/action_selection.rs
+++ /dev/null
@@ -1,8 +0,0 @@
-//! Contains the action selection component.
-
-use leptos::prelude::*;
-
-#[component]
-pub fn ActionSelection() -> impl IntoView {
-    view! {}
-}
diff --git a/src/pages/loaner_kiosk/loaner_kiosk.rs b/src/pages/loaner_kiosk/loaner_kiosk.rs
deleted file mode 100644
index c2f2f4e..0000000
--- a/src/pages/loaner_kiosk/loaner_kiosk.rs
+++ /dev/null
@@ -1,39 +0,0 @@
-//! This defines the base component of the Chromebook loaner kiosk application.
-//! There are going to be 3 definitive screens that the users are able to see and use:
-//!  - The username entry screen: users will enter their name and email.
-//!  - The action selection screen: users will be given a choice regarding their loaner status.
-//!  - The completion screen: users will be given a success message should their information save.
-//! Refer to each component's file for more information about how they work.
-
-use super::{action_selection::ActionSelection, user_entry::UserEntry};
-use crate::common::LoanerUser;
-use leptos::prelude::*;
-
-/// Used to keep track of the currently displayed controls.
-#[derive(Clone, Copy, PartialEq)]
-pub enum LoanerKioskState {
-    UserEntry,
-    ActionSelection,
-    Complete,
-}
-
-/// The base Chromebook loaner kiosk component.
-///
-/// Controls the state of the loaner application and gives a space for the subcomponents to display.
-#[component]
-pub fn LoanerKiosk() -> impl IntoView {
-    let current_state = RwSignal::new(LoanerKioskState::UserEntry);
-    let user = RwSignal::new(LoanerUser::new());
-
-    view! {
-        <div class="flex w-screen h-screen justify-center items-center">
-            <UserEntry
-                on_submit={move || { current_state.set(LoanerKioskState::ActionSelection); }}
-                user={user}
-                shown={current_state.get() == LoanerKioskState::UserEntry}
-            />
-            <ActionSelection
-            />
-        </div>
-    }
-}
diff --git a/src/pages/loaner_kiosk/mod.rs b/src/pages/loaner_kiosk/mod.rs
deleted file mode 100644
index 2dfbb58..0000000
--- a/src/pages/loaner_kiosk/mod.rs
+++ /dev/null
@@ -1,3 +0,0 @@
-mod action_selection;
-mod loaner_kiosk;
-mod user_entry;
diff --git a/src/pages/loaner_kiosk/user_entry.rs b/src/pages/loaner_kiosk/user_entry.rs
deleted file mode 100644
index 17e62ae..0000000
--- a/src/pages/loaner_kiosk/user_entry.rs
+++ /dev/null
@@ -1,72 +0,0 @@
-//! Contains the user entry component.
-//! It would be preferred if the user was restricted to a real user account. This is the URL:
-//! https://admin.googleapis.com/admin/directory/v1/users?domain=region15.org&projection=BASIC&query=isArchived=false%20isAdmin=false
-//! Once they select this, they can then check in/out the loaners under their account.
-//! That means that the Leptos server will now REQUIRE a user token of some sort, probably
-//! a service account. That's going to be a pain to set up.
-//! For example's sake here, we're just going to take a first name, last name, and email address.
-
-use crate::common::LoanerUser;
-use crate::components::{ActionButton, OutlinedTextField};
-use leptos::leptos_dom::logging::console_log;
-use leptos::prelude::*;
-
-/// The user entry component for the loaner kiosk.
-///
-/// Users are able to input their first name, last name, and email address. In the future, this
-/// component should call out to a server function that allows the application to check if the
-/// information provided is correct. This may be by creating storing a series of user records
-/// or simply querying Google's Admin API to check if the user is valid.
-#[component]
-pub fn UserEntry(
-    /// A callback function used when the control has determined this stage is finished.
-    #[prop()]
-    mut on_submit: impl FnMut() + 'static,
-    /// The global user record.
-    #[prop()]
-    user: RwSignal<LoanerUser>,
-    /// Whether this control is shown. Required for use with CSS animations.
-    #[prop()]
-    shown: bool,
-) -> impl IntoView {
-    let disabled = RwSignal::new(false);
-    let on_click = move |_| {
-        disabled.set(true);
-        on_submit();
-        console_log("Clicked!");
-        disabled.set(false);
-    };
-
-    view! {
-        <div
-            class="transition-all duration-300 flex flex-col"
-            class=(["opacity-100 delay-300"], shown)
-            class=(["opacity-0"], !shown)
-        >
-            <OutlinedTextField
-                label="First Name".into()
-                placeholder="John".into()
-                disabled={disabled}
-                value=RwSignal::new("".to_owned())
-            />
-            <OutlinedTextField
-                label="Last Name".into()
-                placeholder="Smith".into()
-                disabled={disabled}
-                value=RwSignal::new("".to_owned())
-            />
-            <OutlinedTextField
-                label="Region 15 Email".into()
-                placeholder="jsmith@region15.org".into()
-                disabled={disabled}
-                value=RwSignal::new("".to_owned())
-            />
-            <ActionButton
-                on:click=on_click
-                disabled={disabled}
-            >
-                "Submit"
-            </ActionButton>
-        </div>
-    }
-}
diff --git a/src/pages/mod.rs b/src/pages/mod.rs
index 1650220..47a8092 100644
--- a/src/pages/mod.rs
+++ b/src/pages/mod.rs
@@ -1,6 +1,5 @@
 mod about_page;
 mod home_page;
-mod loaner_kiosk;
 mod provider_entry;
 
 pub use self::{about_page::*, home_page::*, provider_entry::*};