Skip to content

Add search-shopify-liquid-documentation extension#8873

Merged
raycastbot merged 5 commits intoraycast:mainfrom
maximedaraize:ext/search-shopify-liquid-documentation
Nov 22, 2023
Merged

Add search-shopify-liquid-documentation extension#8873
raycastbot merged 5 commits intoraycast:mainfrom
maximedaraize:ext/search-shopify-liquid-documentation

Conversation

@maximedaraize
Copy link
Contributor

@maximedaraize maximedaraize commented Oct 19, 2023

Description

Search Shopify's Liquid Documentation Directly Raycast.

The four main categories of the liquid documentation include:

  • Basic
  • Operators
  • Filters
  • Objects

Here is the Shopify's liquid doc as refernce.

Screencast

display the list of Shopify's liquid documentation

display filtering the list when typing

Checklist

@raycastbot raycastbot added the new extension Label for PRs with new extensions label Oct 19, 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.

- change icon size + run fix-lint command
- initial commit
- add screenshot metadata
- reorder search list sections
@pernielsentikaer
Copy link
Collaborator

Hi 👋

How is this compared with this extension by @sandypockets?

@pernielsentikaer pernielsentikaer self-assigned this Oct 20, 2023
@maximedaraize
Copy link
Contributor Author

Hi 👋

How is this compared with this extension by @sandypockets?

Hi @pernielsentikaer 👋🏻

@sandypockets extension reference the Shopify's changelog (newest api's, releases, etc). I have it installed, pretty good way to know about latest changes 😉

The extension I am submitting reference the pages of the official liquid documentation of Shopify. It gives you a quick access to go to the docs directly from Raycast.

Both extension are related to Shopify, but do not provide the same information. I hope this clarify the difference between both.

@pernielsentikaer
Copy link
Collaborator

Would it make sense to merge them into one Shopify extension instead of having two?

@maximedaraize
Copy link
Contributor Author

Would it make sense to merge them into one Shopify extension instead of having two?

I like the idea of having different extension with different purposes. Also, what I appreciate about having separate extensions is being responsible for its maintenance.

If I understand correctly, by forking an existing extension and contributing to it, the original author would need to accept the changes and any future changes related to bug fixes and/or new add-on?

But if we were to merge them into one, I should follow this Contribute to an Extension guide?

@maximedaraize
Copy link
Contributor Author

Hi @pernielsentikaer please let know what should be the next step. I would highly appreciate your feedbacks.

Thank you

@raycastbot
Copy link
Collaborator

This pull request has been automatically marked as stale because it did not have any recent activity.

It will be closed if no further activity occurs in the next 7 days to keep our backlog clean 😊

@raycastbot raycastbot added the status: stalled Stalled due inactivity label Nov 19, 2023
@raycastbot raycastbot removed the status: stalled Stalled due inactivity label Nov 22, 2023
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 bf30ca4 into raycast:main Nov 22, 2023
@raycastbot
Copy link
Collaborator

Published to the Raycast Store:
https://raycast.com/maximedaraize/search-shopify-liquid-documentation

@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

new extension Label for PRs with new extensions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants