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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(graph): handle special characters in url parameters #13836

Merged
merged 1 commit into from Dec 15, 2022

Conversation

philipjfulcher
Copy link
Collaborator

Current Behavior

  • focusing or tracing a project with a special character (such as @redwoodjs/cli) will cause an error screen

Expected Behavior

  • focusing or tracing a project with a special character (such as @redwoodjs/cli) works normally

@vercel
Copy link

vercel bot commented Dec 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nx-dev ✅ Ready (Inspect) Visit Preview Dec 14, 2022 at 11:36PM (UTC)

Copy link
Contributor

@meeroslav meeroslav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the quick fix! Looks good to me

@meeroslav meeroslav merged commit 9163756 into nrwl:master Dec 15, 2022
@philipjfulcher philipjfulcher deleted the fix-project-slashes-in-url branch December 19, 2022 15:38
FrozenPandaz pushed a commit that referenced this pull request Dec 19, 2022
Co-authored-by: Philip Fulcher <philipfulcher@Philips-MacBook-Air.local>
(cherry picked from commit 9163756)
@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants