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

Bug: New insight repo form is case sensitive #734

Closed
1 of 2 tasks
bdougie opened this issue Dec 23, 2022 · 2 comments
Closed
1 of 2 tasks

Bug: New insight repo form is case sensitive #734

bdougie opened this issue Dec 23, 2022 · 2 comments
Labels
🐛 bug Something isn't working
Milestone

Comments

@bdougie
Copy link
Member

bdougie commented Dec 23, 2022

Describe the bug

Discovered by a user in our discussions

open-sauced/community#2 (reply in thread)

Hiya Brian @bdougie It still does not show up and takes me back to this discussion page to ask for it to be added.

Do you mind checking please after the holidays.

Thanks, Warren 😄

We should make this not case sensitive.

Steps to reproduce

IMG_0208

  1. navigate to insights hub
  2. create insights page
  3. add umbraco/umbraco-cms - it doesn't work
  4. Now add umbraco/Umbraco-CMS to another new page - it works

Affected services

insights.opensauced.pizza

Platforms

No response

Browsers

No response

Environment

No response

Additional context

the filtering on the repo is not sensitive. IMG_0207

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@bdougie bdougie added 🐛 bug Something isn't working 👀 needs triage and removed 👀 needs triage labels Dec 23, 2022
@bdougie bdougie added this to the Papercuts milestone Dec 23, 2022
@0-vortex
Copy link
Contributor

While the frontend should search for case-insensitive repos, the backend should mirror the github api and provide the same functionality; for that purpose opened up an issue in the public backend: https://github.com/open-sauced/api.opensauced.pizza/issues/76

It's not blocking coding current issue but it does block testing it.

@brandonroberts
Copy link
Contributor

brandonroberts commented Jan 17, 2023

This has been resolved in the latest release of the API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
No open projects
Development

No branches or pull requests

3 participants