-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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 mutedeck extension #16432
base: main
Are you sure you want to change the base?
Add mutedeck extension #16432
Conversation
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. |
There was a problem hiding this 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:
-
You have a lot of dot files; are they necessary?
-
Could you please use the standard
.prettierrc
settings -
Could you add at least one screenshot of the extension's view commands
-
What is the plan with the screenshots in the assets folder?
-
Do you have a screencast of the extension in use so I can see the flow?
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.
Thanks for the review @pernielsentikaer! I've addressed all the feedback:
Changes made:
The extension is now working correctly and follows all Raycast standards. Please let me know if you'd like to see any other improvements! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking your time to correct this 🙂
A few more:
- I can see all dotfiles still, should we delete them from the PR?
- metadata images should be in a folder called
metadata
and be takign with our screenshot tool
Thanks for the additional feedback! I've addressed both points:
Please let me know if you'd like to see any other improvements or if there's anything else I can clarify! |
Did you push this into |
Yes, I've now pushed all the changes to the
Sorry for the branch confusion earlier. Everything should be in the right place now! |
There is no new pushes here: https://github.com/chadrwalters/raycast-extensions/tree/ext/mutedeck |
4b8af8d
to
ee14f35
Compare
@pernielsentikaer First, I want to sincerely thank you for your patience and guidance throughout this PR process. I really appreciate the time you've taken to help me get this right. I should mention - this is actually my first PR to a public repository in about 10 years! I've been away from production code for a while, and I chose Raycast for my return because it's one of my favorite tools. Thank you for bearing with me as I get back into the swing of things. I've now addressed all the feedback:
All changes have been pushed to the ext/mutedeck branch. You can verify the changes at the latest commits (ebf71fa). Thanks again for helping me make this contribution as polished as possible! |
Description
MuteDeck Extension for Raycast
This extension provides quick access to MuteDeck controls directly from Raycast, allowing users to manage their meeting presence efficiently.
Features
Key Highlights
Technical Details
Security
Configuration
Users can customize:
This extension helps streamline meeting controls for MuteDeck users while maintaining a focus on reliability and user safety.
Screencast
Checklist
npm run build
and tested this distribution build in Raycastassets
folder are used by the extension itselfREADME
are placed outside of themetadata
folder