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

Update Monday extension - View items in a board and open in browser + Raycast hooks to simplify caching #16746

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

Conversation

xmok
Copy link
Contributor

@xmok xmok commented Jan 31, 2025

Description

  • Add metadata image
  • Add CHANGELOG
  • Add extension category
  • Action to "Open Extension Preferences" in ErrorView
  • View items in a board and open in browser

Dev Changes

  • useForm in Add for better control and validation
  • useCachedPromise and usePomise to reduce useEffect
  • useCachedPromise also reduces need for in-house caching through LocalStorage (some commands are still using LocalStorage for now)

NOTE: To keep the PR manageable, not all previous caching via LocalStorage has been replaced with the new hooks. Future PRs can handle it. Similarly, the error handling can be tweaked but would extend this PR.

Screencast

"View Board Items" Action is new otherwise the screencast is simply showing that nothing was broken through the hooks:

monday-250201.mp4

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: monday Issues related to the monday extension labels Jan 31, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Jan 31, 2025

Thank you for your contribution! 🎉

🔔 @freak4pc 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.

You can expect an initial review within five business days.

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

Successfully merging this pull request may close these issues.

2 participants