Skip to content

Add copyq extension#9139

Closed
andrewcincotta wants to merge 2 commits intoraycast:mainfrom
andrewcincotta:ext/copyq
Closed

Add copyq extension#9139
andrewcincotta wants to merge 2 commits intoraycast:mainfrom
andrewcincotta:ext/copyq

Conversation

@andrewcincotta
Copy link
Contributor

@andrewcincotta andrewcincotta commented Nov 7, 2023

Description

Screencast

Checklist

- Changed extension icon, changed package.json a bit, fixed preferences in index.tsx
- Created version 1 of Raycast extension. Offers preferences to set path to copyQ, default tab, and default number of items. - Removed Python scripts and rewrote in TypeScript.
- Added copyq-clipboard-manager, a raycast extension template, as well as the utils folder with the python scripts to be used by the extension.
- Update README.md
- Create README.md
- Initial commit
@raycastbot raycastbot added the new extension Label for PRs with new extensions label Nov 7, 2023
@raycastbot
Copy link
Collaborator

Congratulation on your new Raycast extension! 🚀

We will review it shortly. Once the PR is approved and merged, the extension will be available on the Store.

- Fixed ESLint so that extension will pass review.
- Changed extension icon, changed package.json a bit, fixed preferences in index.tsx
- Created version 1 of Raycast extension. Offers preferences to set path to copyQ, default tab, and default number of items. - Removed Python scripts and rewrote in TypeScript.
- Added copyq-clipboard-manager, a raycast extension template, as well as the utils folder with the python scripts to be used by the extension.
- Update README.md
- Create README.md
- Initial commit
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:

  • Could you add metadata images

  • Could you try to set the default path instead of letting user do it, this way they only have to change it if it's not in the default path - this could make the preference default:false

  • If you set a wrong path, then the extension crashes, could you handle that gracefully?

copyq 2023-11-10 at 07 15 34

  • Some of the default prettier files are missing, could you use the command Create Extension and copy all the missing files in root from the temporary extension 🙂

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.

Comment on lines +13 to +17
interface Preferences {
copyq_path: string;
default_tab: string;
default_num_items: number;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
interface Preferences {
copyq_path: string;
default_tab: string;
default_num_items: number;
}

Raycast autogenerate this

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