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

feat: repos in search #170

Merged
merged 3 commits into from
Nov 30, 2024
Merged

feat: repos in search #170

merged 3 commits into from
Nov 30, 2024

Conversation

zugdev
Copy link
Contributor

@zugdev zugdev commented Nov 29, 2024

Resolves #169

QA:

image

image

@zugdev zugdev requested a review from 0x4007 as a code owner November 29, 2024 17:04
@ubiquity-os-deployer
Copy link

ubiquity-os-deployer bot commented Nov 29, 2024

@zugdev
Copy link
Contributor Author

zugdev commented Nov 29, 2024

@0x4007 I've added org scoring to the search too.

Previously:
image

Now:
image

The weight to repo and org searches is the lowest of them all, so search still prioritizes body, title and metadata.

@zugdev
Copy link
Contributor Author

zugdev commented Nov 29, 2024

Multi-term search is also enabled:

image

@0x4007
Copy link
Member

0x4007 commented Nov 30, 2024

https://404ad59a.devpool-directory-ui.pages.dev/ubiquity-os-marketplace?search=Text-vector-embeddings

image

  • Yeah let's do rounded avatar. I suppose it might look best to match the user avatar to the org avatar on the left in terms of width, height, and placement.
  • check if you can vertical align center on the devpool directory title. Looks a bit low compared to the org icon on the top left
  • disable auto capitalization and spell check in the text search box

@0x4007 0x4007 merged commit b2efa02 into ubiquity:development Nov 30, 2024
1 of 2 checks passed
@0x4007 0x4007 mentioned this pull request Nov 30, 2024
@ubiquity-os ubiquity-os bot mentioned this pull request Dec 2, 2024
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.

add a repo searching scorer
2 participants