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 lingo-rep-raycast extension #16512

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

Conversation

asashay
Copy link

@asashay asashay commented Jan 20, 2025

Description

The LingoRep Raycast extension makes it easy to translate text between languages. Anyone can use it for quick translations. Once you sign in, you can save your translations and use the spaced repetition algorithm in the web app to help memorize them effectively. More about it on the web site - https://lingorep.com/

Screencast

Screencast video

Checklist

- fix warnings
- fix lint errors
- prepare prod build
@raycastbot raycastbot added the new extension Label for PRs with new extensions label Jan 20, 2025
@raycastbot
Copy link
Collaborator

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.

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 make a screencast of the flow so I can see how it works 😊

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.

extensions/lingo-rep-raycast-extension/CHANGELOG.md Outdated Show resolved Hide resolved
extensions/lingo-rep-raycast-extension/package.json Outdated Show resolved Hide resolved
@pernielsentikaer pernielsentikaer self-assigned this Jan 24, 2025
@asashay
Copy link
Author

asashay commented Jan 26, 2025

@pernielsentikaer thank you for getting back to me! I've added screencast and description - please check them out

Comment on lines 24 to 35
"copy:shared-packages": "./copy-shared-packages.sh",
"watch:translate": "chokidar '../../packages/translate/dist/**/*' -c 'npm run copy:shared-packages'",
"watch:lang-options": "chokidar '../../packages/lang-options/dist/**/*' -c 'npm run copy:shared-packages'",
"watch:shared-packages": "npm-run-all --parallel watch:translate watch:lang-options",
"npm:install": "npm install --prefix /Users/alex/Work/Projects/alex/lingo-rep-monorepo/apps/lingo-rep-raycast-extension",
"build": "npm run npm:install && ray build -e dist",
"dev:raycast": "ray develop",
"dev": "npm run npm:install && npm run copy:shared-packages && npm-run-all --parallel watch:shared-packages dev:raycast",
"dev-raycast": "",
"fix-lint": "ray lint --fix",
"lint": "ray lint",
"publish": "npx @raycast/api@latest publish"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could we just have the default ones?

Copy link
Author

Choose a reason for hiding this comment

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

@pernielsentikaer changed, please take a look

@asashay asashay changed the title Add lingo-rep-raycast-extension extension Add lingo-rep-raycast extension Feb 4, 2025
@asashay
Copy link
Author

asashay commented Feb 4, 2025

hey @pernielsentikaer , I also shortened my repo URL from lingo-rep-raycast-extension.git to lingo-rep-raycast.git. I think it makes more sense from the readability stand point.

Will it be ok to publish it like that? Won't it cause issues when I submit new versions?
Is it better to close this one and resubmit it maybe?

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