freshlybakedca.ke / patisserie
Your one-stop-cake-shop for everything Freshly Baked has to offer
fork atom

feat(m/direct)!: make links case-insensitive #149

merged opened by a.starrysky.fyi targeting main from private/minion/push-prroxtvotrzn

It's not good if our links are case sensitive as different capitalizations aren't that distinct really. It's still important to make sure we don't mess up capitalization in the UI or when forwarding to search engines/etc., though

BREAKING-CHANGE: If you're using links that were made with capitals, those links will stop working

STACK 1
#149 feat(m/direct)!: make links case-insensitive
0
#0
Labels

None yet.

requested-reviewers

None yet.

approved

None yet.

tested-working

None yet.

rejected

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:uuyqs6y3pwtbteet4swt5i5y/sh.tangled.repo.pull/3mbmzcz4krf22
+2 -2
Diff #0
unified split
+2 -2
menu/src/main.rs
··· 50 } 51 52 async fn get_redirect(default_location: &str, go: &str) -> Redirect { 53 - let redirect = sqlx::query!(r#"SELECT ("to") FROM direct WHERE "from" = $1 LIMIT 1"#, go) 54 .fetch_one( 55 STATE 56 .get() ··· 134 135 let create_call = sqlx::query!( 136 r#"INSERT INTO direct ("from", "to", "owner") VALUES ($1, $2, $3)"#, 137 - create.from, 138 create.to, 139 owner, 140 )
··· 50 } 51 52 async fn get_redirect(default_location: &str, go: &str) -> Redirect { 53 + let redirect = sqlx::query!(r#"SELECT ("to") FROM direct WHERE "from" = $1 LIMIT 1"#, go.to_lowercase()) 54 .fetch_one( 55 STATE 56 .get() ··· 134 135 let create_call = sqlx::query!( 136 r#"INSERT INTO direct ("from", "to", "owner") VALUES ($1, $2, $3)"#, 137 + create.from.to_lowercase(), 138 create.to, 139 owner, 140 )

History

1 round 0 comments
sign up or login to add to the discussion
a.starrysky.fyi submitted #0
1 commit
expand
800c2e53
feat(m/direct)!: make links case-insensitive
expand 0 comments
pull request successfully merged