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

Add early-tools-news extension #16169

Closed
wants to merge 6 commits into from

Conversation

thejulez
Copy link

@thejulez thejulez commented Jan 4, 2025

Description

Screencast

Checklist

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

raycastbot commented Jan 4, 2025

Congratulations on your new Raycast extension! 🚀

You can expect an initial review within five business days.

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

@pernielsentikaer
Copy link
Collaborator

Hi 👋

Thanks for your contribution 🔥

Could you look into the CI errors mentioned here so I can review it

@pernielsentikaer pernielsentikaer self-assigned this Jan 12, 2025
- Final prettification.
- Fixed CLI error.
@thejulez
Copy link
Author

thejulez commented Jan 14, 2025 via email

const [state, setState] = useState<State>({ items: [], isLoading: true });
const [searchText, setSearchText] = useState("");

useEffect(() => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can maybe use useCachedPromise from @raycast/utils here

Copy link
Author

Choose a reason for hiding this comment

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

Good idea! Does this stop the plugin from being released on the store? Happy to make the changes, just need to know :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

You can indeed make the change, when you publish it just updates the PR

@thejulez thejulez closed this Jan 16, 2025
@thejulez thejulez reopened this Jan 16, 2025
@thejulez thejulez closed this Jan 16, 2025
@thejulez thejulez deleted the ext/early-tools-news branch January 16, 2025 14:08
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