Skip to content

Commit 2ee62ef

Browse files
committed
refactor: fix clippy warnings
1 parent a3e9285 commit 2ee62ef

5 files changed

Lines changed: 21 additions & 10 deletions

File tree

src/pages/flashcards/detail/mod.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ pub fn DeckDetailPage() -> impl IntoView {
117117
};
118118

119119
leptos::task::spawn_local(async move {
120-
if let Ok(_) = update_deck(id, name, description).await {
120+
if update_deck(id, name, description).await.is_ok() {
121121
deck_resource.refetch();
122122
show_rename_modal.set(false);
123123
}
@@ -143,7 +143,7 @@ pub fn DeckDetailPage() -> impl IntoView {
143143
if confirmed {
144144
let project_id = deck.project_id;
145145
leptos::task::spawn_local(async move {
146-
if let Ok(_) = delete_deck(id).await {
146+
if delete_deck(id).await.is_ok() {
147147
leptos_router::hooks::use_navigate()(
148148
&format!("/projects/{}/decks", project_id),
149149
Default::default(),
@@ -157,7 +157,7 @@ pub fn DeckDetailPage() -> impl IntoView {
157157
{
158158
let project_id = deck.project_id;
159159
leptos::task::spawn_local(async move {
160-
if let Ok(_) = delete_deck(id).await {
160+
if delete_deck(id).await.is_ok() {
161161
leptos_router::hooks::use_navigate()(
162162
&format!("/projects/{}/decks", project_id),
163163
Default::default(),
@@ -350,7 +350,7 @@ pub fn DeckDetailPage() -> impl IntoView {
350350
// Generation Modal
351351
<Show when=move || show_generate_modal.get()>
352352
<GenerationModal
353-
deck_id=Signal::derive(move || deck_id())
353+
deck_id=Signal::derive(deck_id)
354354
files=Signal::derive(move || {
355355
project_files_resource.get()
356356
.flatten()

src/pages/flashcards/viewer.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,6 @@ fn FlashcardViewer(
230230
<div class="rounded-2xl border border-slate-800 bg-slate-900/50 p-8">
231231
{move || match card.get() {
232232
Some(card) => {
233-
let on_toggle = on_toggle.clone();
234233
view! {
235234
<div class="space-y-8">
236235
<div class="min-h-[200px]">

src/pages/invites.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,10 @@ pub fn AdminInvitesPage() -> impl IntoView {
9191
</>
9292
}.into_any(),
9393
Some(Err(e)) => view! { <> <p>{e.to_string()}</p> </> }.into_any(),
94-
None => view! { <> </> }.into_any(),
94+
None => {
95+
let _: () = view! { <> </> };
96+
().into_any()
97+
},
9598
}}}
9699
</Show>
97100
</Show>

src/pages/projects/detail.rs

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -311,13 +311,19 @@ pub fn ProjectDetailPage() -> impl IntoView {
311311
Some(Err(err)) => view! {
312312
<p class="text-sm text-rose-300">{err.to_string()}</p>
313313
}.into_any(),
314-
None => view! { <> </> }.into_any(),
314+
None => {
315+
let _: () = view! { <> </> };
316+
().into_any()
317+
},
315318
}}}
316319
</Show>
317320
</div>
318321
}.into_any(),
319322
Some(Err(err)) => view! { <> <p class="text-sm text-rose-300">{err.to_string()}</p> </> }.into_any(),
320-
None => view! { <> </> }.into_any(),
323+
None => {
324+
let _: () = view! { <> </> };
325+
().into_any()
326+
},
321327
}}}
322328
</Show>
323329
</Show>
@@ -420,7 +426,7 @@ fn format_number(num: i64) -> String {
420426
let chars: Vec<char> = s.chars().collect();
421427

422428
for (i, ch) in chars.iter().enumerate() {
423-
if i > 0 && (chars.len() - i) % 3 == 0 {
429+
if i > 0 && (chars.len() - i).is_multiple_of(3) {
424430
result.push(',');
425431
}
426432
result.push(*ch);

src/pages/projects/list.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,10 @@ pub fn ProjectsPage() -> impl IntoView {
8989
Some(Err(err)) => view! {
9090
<p class="text-sm text-rose-300">{err.to_string()}</p>
9191
}.into_any(),
92-
None => view! { <> </> }.into_any(),
92+
None => {
93+
let _: () = view! { <> </> };
94+
().into_any()
95+
},
9396
}}}
9497
</Show>
9598
</div>

0 commit comments

Comments
 (0)