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

Ext/ipinfo extension #16201

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

Conversation

wzfukui
Copy link

@wzfukui wzfukui commented Jan 7, 2025

Description

This is a Raycast extension for ipinfo.io. With this extension you can lookup detailed information about any IPv4/IPv6 address.

Screencast

quickview-how-to-query-ipinfo quickview-how-to-set-apitoken

Checklist

@raycastbot raycastbot added the new extension Label for PRs with new extensions label Jan 7, 2025
@raycastbot
Copy link
Collaborator

Congratulations on your new Raycast extension! 🚀

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

Once the PR is approved and merged, the extension will be available on our Store.

Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Hi 👋

Thanks for your contribution 💪

I have now tested your extension, and I have some feedback ready for you:

  • Token should only be set in preferences, not a view in the extension

  • Could you add at least one screenshot of the extension

  • Maybe add an argument for IP instead of inline in the search field?

I'm looking forward to testing this extension again 🔥

Request a new review when you are ready. Feel free to contact me here or at Slack if you have any questions.

@pernielsentikaer pernielsentikaer self-assigned this Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new extension Label for PRs with new extensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants