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 key value extension to get the json from remote URL #12187

Merged

Conversation

NirLevy98
Copy link
Contributor

@NirLevy98 NirLevy98 commented May 5, 2024

Description

Improve the key-value extension to load the new key value from remote url

Screencast

Screen.Recording.2024-05-05.at.13.53.12.mov

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: key-value Issues related to the key-value extension OP is author The OP of the PR is the author of the extension labels May 5, 2024
@NirLevy98 NirLevy98 force-pushed the update-key-value-to-use-remote-url branch from cd649b2 to 6708643 Compare May 5, 2024 11:04
@NirLevy98 NirLevy98 changed the title Update key value extention to get the json from remote URL Update key value extension to get the json from remote URL May 5, 2024
@NirLevy98 NirLevy98 force-pushed the update-key-value-to-use-remote-url branch 2 times, most recently from 87d8407 to 12db2d4 Compare May 5, 2024 11:08
@NirLevy98 NirLevy98 force-pushed the update-key-value-to-use-remote-url branch from 12db2d4 to fd9dff0 Compare May 5, 2024 11:09
@raycastbot raycastbot added the OP is contributor The OP of the PR is a contributor of the extension label May 6, 2024
@raycastbot
Copy link
Collaborator

raycastbot commented May 6, 2024

Thank you for your contribution! 🎉

🔔 @NirLevy98 @gkpln3 you might want to have a look.

@gkpln3
Copy link
Contributor

gkpln3 commented May 6, 2024

Lgtm

Comment on lines +39 to +59
"name": "jsonURLPath",
"description": "URL path to a json file with key value pairs",
"type": "textfield",
"required": true,
"title": "Key Value Json URL"
},
{
"name": "jsonKeyForKey",
"description": "Key name to search for",
"type": "textfield",
"required": true,
"title": "Key"
},
{
"name": "jsonKeyForValue",
"description": "Key name to get the value from",
"type": "textfield",
"required": true,
"title": "Value"
}
]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should the new preferences be required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's should be required only under the reload command

Copy link
Collaborator

Choose a reason for hiding this comment

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

Makes sense 🙂

@pernielsentikaer pernielsentikaer self-assigned this May 6, 2024
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 032470a into raycast:main May 6, 2024
6 checks passed
Copy link
Contributor

github-actions bot commented May 6, 2024

Published to the Raycast Store:
https://raycast.com/NirLevy/key-value

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

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

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: key-value Issues related to the key-value extension OP is author The OP of the PR is the author of the extension OP is contributor The OP of the PR is a contributor of the extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants