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

[dotNew] update extension API #17453

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[dotNew] update extension API #17453

wants to merge 3 commits into from

Conversation

ViGeng
Copy link
Contributor

@ViGeng ViGeng commented Feb 28, 2025

Description

I was trying to add a category filter in the search bar mentioned in #17169. However, the metadata can not support this.

So, I updated the API in passing.

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: dot-new Issues related to the dot-new extension labels Feb 28, 2025
@raycastbot
Copy link
Collaborator

Thank you for your contribution! 🎉

🔔 @natterstefan @xmok you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

Due to our current reduced availability, the initial review may take up to 10-15 business days

Copy link
Contributor

@xmok xmok left a comment

Choose a reason for hiding this comment

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

Feel free to add yourself as a contributor in "package.json" if interested.

And re: metadata, you can modify the script:
right now it flattens the tables but you could choose to keep the tables separate, map them into a flattened Array alongside the category then we could use the category for a filter!

@@ -41,7 +31,7 @@ export default function main() {
key={index}
title={url}
subtitle={text}
icon={getFaviconUrl(url)}
icon={getFavicon("https://" + url)}
Copy link
Contributor

Choose a reason for hiding this comment

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

For consistency, can we keep the same fallback?
fallback: Icon.Globe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension: dot-new Issues related to the dot-new extension extension fix / improvement Label for PRs with extension's fix improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants