Skip to content

[Easy Variable] Enhancements and tweaks#17551

Merged
raycastbot merged 2 commits intoraycast:mainfrom
LitoMore:easyvariable
Mar 5, 2025
Merged

[Easy Variable] Enhancements and tweaks#17551
raycastbot merged 2 commits intoraycast:mainfrom
LitoMore:easyvariable

Conversation

@LitoMore
Copy link
Contributor

@LitoMore LitoMore commented Mar 5, 2025 •
edited
Loading

Description

  • Reformat project profile files with Prettier
  • Use password field for API keys and secrets
  • Add Raycast Pro subscription hint
  • Remove unused pnpm configurations
  • Fix the author's Raycast ID

Checklist

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

raycastbot commented Mar 5, 2025 •
edited
Loading

Thank you for your contribution! 🎉

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

Due to our current reduced availability, the initial review may take up to 10-15 business days

greptile-apps[bot]
greptile-apps bot reviewed Mar 5, 2025
View reviewed changes
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR enhances the Easy Variable extension with security improvements and code organization updates, focusing on preference handling and API key security.

  • Improved security by changing API key fields from textfield to password type in package.json
  • Removed unnecessary preferences.ts file since Preferences interface is auto-generated in raycast-env.d.ts
  • Added proper Raycast Pro subscription check in raycast.ts using environment.canAccess(AI)
  • Fixed typo in README.md where "nabled" should be "enabled" on line 16
  • Consider using showFailureToast from @raycast/utils instead of manual error handling in translator modules

11 file(s) reviewed, 10 comment(s)
Edit PR Review Bot Settings | Greptile

@LitoMore LitoMore force-pushed the easyvariable branch 3 times, most recently from 8ebd6c6 to bd72a74 Compare March 5, 2025 02:10
@pernielsentikaer pernielsentikaer self-assigned this Mar 5, 2025
pernielsentikaer
pernielsentikaer approved these changes Mar 5, 2025
View reviewed changes
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 👋

Looks good to me, approved 🔥

@raycastbot raycastbot merged commit 5103bd9 into raycast:main Mar 5, 2025
2 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2025

Published to the Raycast Store:
https://raycast.com/keleus/easyvariable

@raycastbot
Copy link
Collaborator

raycastbot commented Mar 5, 2025

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

@LitoMore LitoMore deleted the easyvariable branch March 5, 2025 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pernielsentikaer pernielsentikaer pernielsentikaer approved these changes

+1 more reviewer

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@pernielsentikaer pernielsentikaer

Labels

extension: easyvariable Issues related to the easyvariable extension extension fix / improvement Label for PRs with extension's fix improvements

Milestone

No milestone