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

Remove legacy shop functionality and replace with simpler approach #284

Open
1 of 3 tasks
mischacolley opened this issue Mar 30, 2020 · 2 comments
Open
1 of 3 tasks

Comments

@mischacolley
Copy link
Contributor

mischacolley commented Mar 30, 2020

  • Remove legacy Shopify code. Link direct to shop.aimementoring.com/[product] for now as we are on /the-mentor
  • Create Blueprint product component
  • Integrate Blueprint product component with Contentful so products can be linked to in pages and /stories

As discussed in #32

Hmm my initial thought was as @camposcristianeze said ... I'd like to keep the Shopify cart functionality around for products like the Book etc. But given we are trying to simplify logic directly on the website I'm also open to discussion if there is a way we could essentially keep the same feel for the user minus the maintenance cost discussed above.

i.e. I have considered a few causes where using the below well could be ok:

At it's simplest a blog post or piece of content could simply link to shop.aimementoring.com/product or you could direct load products into the cart like this https://help.shopify.com/en/themes/customization/cart/use-permalinks-to-preload-cart

Adding slightly more complexity, but providing the ability to content manage and pull in Product images etc, you could of-load the integration work to Contentful using https://www.contentful.com/developers/marketplace/contentful-ui-shopify/ and then if we ever wanted to pull product details for any integrations we do so from one API/Integration. Combine with the above link destination of product single or cart and you have pretty similar functionality.

@charliemckenzie @lulen11 @kbardi @camposcristianeze @bronz3beard open to discussion ...

And as Cris said:

I like this but with a slightly different approach...

I would say remove all the "shop-like" integration (all the scripts querying to Shopify, /products, etc) and just leave one/two products that are not in shop.aimementoring.com with custom "Campaign" pages coming straight from Contentful.

This might also need a Blueprint Component so then can use it in other contexts :) but the groundwork could be made here and then transferred 👍

@camposcristian
Copy link
Contributor

I like this approach! Simpler and less error prone.

@charliemckenzie
Copy link
Contributor

Love the contentful approach, makes it simpler and saves using custom images etc.

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

No branches or pull requests

3 participants