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

Blog post for "Remixing Shopify Workshops" #93

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

Conversation

DavidWittness
Copy link

@DavidWittness DavidWittness commented Apr 28, 2023

Added blog post for "Remixing Shopify Workshops"

Content has gone through an editorial pass from the Shopify side, as well as @mjackson.

Feel free to suggest/edit/tweak for clarity, correctness, and formatting.

Also taking suggestions for a better cover photo. I haven't put much thought into it, so I've screenshotted a workshop for now.

Copy link
Member

@MichaelDeBoey MichaelDeBoey left a comment

Choose a reason for hiding this comment

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

Awesome article, I really enjoyed reading how you re-created the app with Remix! 😍

Added some small things that could be improved imo.

P.S.: I'm not part of the core team, but I'm one of the core collaborators on the project. Nice to meet you!

data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Show resolved Hide resolved
data/posts/remixing-shopify-workshops.md Outdated Show resolved Hide resolved
MichaelDeBoey

This comment was marked as duplicate.

DavidWittness and others added 2 commits May 1, 2023 09:41
Will review code specific suggestions in another commit

Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
Removed references to ts and tsx
Updated loaders to return objects in json()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants