keyserj/titles-supporting-spaces-and-dashes

🚀 Help pilot test Ameliorate! Read more

AmeliorateAlpha

Docs

AmeliorateAlpha

Docs

keyserj/titles-supporting-spaces-and-dashes

Basics

Comments

No comments yet!

Quick View

causes

has

causes

has

causes

mitigates

causes

has

causes

has

causes

has

mitigates

has

causes

mitigates

has

causes

Detriment

current titles need to be migrated

Benefit

no need for raw queries

Component

normalize space to dash for uniqueness, e.g. "topic-1"

Detriment

raw SQL which is hard to maintain

Detriment

requires UPDATE to have a SELECT first to ensure unique

Solution

allow dash + space, normalize to dash for URLs + uniqueness

Detriment

confusion if both URLs exist

Detriment

requires pushing back on existing PR

Component

allow spaces in titles

Benefit

clean URLs

Detriment

still should encode URLs everywhere, but don't need to decode

Mitigation

create "findTopicIdByUrlSafeTitle" to isolate code

Benefit

no concern of two titles having the same URL

Component

encode space for URL, e.g. "topic%201"

Component

normalize space to dash for URL, e.g. "topic-1"

Problem

titles aren't human-readable (use dashes instead of spaces)

Component

treat space + dash as different, e.g. "topic-1" and "topic 1" can exist

Detriment

not clean URLs

Solution

allow dash + space, treat them as different but URI-encode space

Mitigation

rely on collation to treat space + dash as identical

Solution

allow spaces and no dashes, normalize to dash for URLs

Component

disallow dashes in titles

Detriment

requires SELECT to use REPLACE to find to correct record

Benefit

trpc and browsers auto-encode spaces

React Flow

Basics

Comments

Welcome to Ameliorate 🔥