Skip to content

#2: case sensitive params#5

Open
spiffytech wants to merge 2 commits intomainfrom
2-case-sensitive-params
Open

#2: case sensitive params#5
spiffytech wants to merge 2 commits intomainfrom
2-case-sensitive-params

Conversation

@spiffytech
Copy link
Copy Markdown
Member

This change fixes #2: captured URL params no longer get transformed to lowercase.

I've also added the repo's first tests. I didn't want to figure out if JSDOM can handle pushstate, so I just set up Playwright to launch Chromium and make assertions in a real browser.

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Tue Dec 6 18:47:34 2022 -0500
#
# On branch 2-case-sensitive-params
# Changes to be committed:
#	new file:   .editorconfig
#	modified:   .gitignore
#	new file:   index.html
#	modified:   package-lock.json
#	modified:   package.json
#	new file:   playwright.config.ts
#	new file:   tests/main.tsx
#	new file:   tests/router.spec.ts
#	modified:   tsconfig.json
#
# Changes not staged for commit:
#	modified:   tests/router.spec.ts
#
@spiffytech spiffytech requested a review from jeswin December 6, 2022 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Router lowercases matched parameters

1 participant