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.

Sign up for GitHub

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

Jump to bottom

Add keywords for SCM to appear via Registry search #5372

Closed
zbuchheit opened this issue Sep 10, 2024 · 2 comments
Closed

Add keywords for SCM to appear via Registry search #5372

zbuchheit opened this issue Sep 10, 2024 · 2 comments
Assignees
@iwahbe
Labels
area/search Concerning search. kind/bug Some behavior is incorrect or out of spec resolution/duplicate This issue is a duplicate of another issue

Comments

@zbuchheit
Copy link
Contributor

Hello!

  • Vote on this issue by adding a 👍 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

Currently, if you search for "SCM", "PaloAlto", or "Palo Alto" in the Registry search, the SCM provider doesn't show as a result

Example

Image

Affected area/feature

Registry Search
@zbuchheit zbuchheit added kind/enhancement Improvements or new features needs-triage Needs attention from the triage team labels Sep 10, 2024
@github-project-automation github-project-automation bot moved this to 🤔 Triage in Docs 📚 Sep 10, 2024
@interurban interurban moved this from 🤔 Triage to 🧳 Backlog in Docs 📚 Sep 25, 2024
@interurban interurban added kind/bug Some behavior is incorrect or out of spec and removed needs-triage Needs attention from the triage team kind/enhancement Improvements or new features labels Sep 25, 2024
@interurban interurban self-assigned this Sep 25, 2024
@interurban interurban added this to the 0.110 milestone Sep 25, 2024
@interurban interurban moved this from 🧳 Backlog to 🎬 Ready in Docs 📚 Sep 25, 2024
@sean1588 sean1588 mentioned this issue Sep 25, 2024
Closed
@interurban interurban mentioned this issue Sep 25, 2024
Open
@sean1588
Copy link
Member

@zbuchheit - our filter on the registry search is actually really basic right now and is mostly based on text matching of the package titles. These titles are actually sourced from the displayName property in the provider's schema file. I notice this provider is bridged, so not sure if you all have control over the display name or if that is set automatically based on some upstream tf provider config. If you have some way to get that updated to contain SCM and Pal Alto, then the search should start working the next time you release the provider.

@interurban interurban modified the milestones: 0.110, 0.111 Oct 1, 2024
@interurban interurban modified the milestones: 0.111, 0.112 Oct 30, 2024
@interurban interurban removed this from the 0.112 milestone Nov 12, 2024
@iwahbe iwahbe added area/search Concerning search. resolution/duplicate This issue is a duplicate of another issue labels Nov 14, 2024
@iwahbe
Copy link
Member

iwahbe commented Nov 14, 2024

Closing as a duplicate of #4902.

@iwahbe iwahbe closed this as completed Nov 14, 2024
@github-project-automation github-project-automation bot moved this from 🎬 Ready to 🏁 Done in Docs 📚 Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iwahbe iwahbe

Labels
area/search Concerning search. kind/bug Some behavior is incorrect or out of spec resolution/duplicate This issue is a duplicate of another issue
Projects
Docs 📚
Status: 🏁 Done
Milestone
No milestone
Development

No branches or pull requests
4 participants
@interurban @sean1588 @iwahbe @zbuchheit