Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[馃悶] trailing slash added to route params #2351

Closed
robisim74 opened this issue Dec 2, 2022 · 3 comments
Closed

[馃悶] trailing slash added to route params #2351

robisim74 opened this issue Dec 2, 2022 · 3 comments
Assignees
Labels
STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working
Milestone

Comments

@robisim74
Copy link
Contributor

Which component is affected?

Qwik City (routing)

Describe the bug

When you have a parameter in the root, not followed by other segments, a trailing slash is added to the parameter:

{ params: { id: '1234/' } }

Reproduction

https://stackblitz.com/edit/qwik-starter-zgnacc

Steps to reproduce

After the app has started, click on Go to id 1234 and look in the Terminal (Ok, just hover the mouse pointer over the button without pressing it... what an amazing improvement is this?!)

Thanks

System Info

System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @builder.io/qwik: 0.15.0 => 0.15.0 
    @builder.io/qwik-city: 0.0.128 => 0.0.128 
    vite: 3.2.4 => 3.2.4

Additional Information

No response

@robisim74 robisim74 added TYPE: bug Something isn't working STATUS-1: needs triage New issue which needs to be triaged labels Dec 2, 2022
@stackblitz
Copy link

stackblitz bot commented Dec 2, 2022

Fix this issue in StackBlitz Codeflow Start a new pull request in StackBlitz Codeflow.

@manucorporat manucorporat added this to the 0.15.0 milestone Dec 2, 2022
@adamdbradley
Copy link
Contributor

Related #2352

@adamdbradley
Copy link
Contributor

Fixed in #2385

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants