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

I know this is a demo but... #12

Open
resistorsoftware opened this issue Aug 5, 2020 · 0 comments
Open

I know this is a demo but... #12

resistorsoftware opened this issue Aug 5, 2020 · 0 comments

Comments

@resistorsoftware
Copy link

resistorsoftware commented Aug 5, 2020

I am curious why this demo chose to brute force graphql instead of using the Shopify API gem?

Seems kind of weird when so much configuration is already baked in for shopify_api... Just curious.

As a consumer of the Shopify App and API gem, and the ecosystem, I deeply underwhelmed by the move to JWT tokens as a thing to solve the broken browser embedded app issue, as put forth so far. Reading the code shows me gaping chasms to cross. I guess this is a group effort and eventually, some effort will result in showing off how everything works together.

It is cool that you show off Apollo for those that use Apollo, and I really liked that you also showed Axios off, as I only use that. So there are hints in this repo as to how it might all work and come together, but it is still not very clear how to architect the combo of the Shopify App with JWT and the Shopify API together. Close. But not quite. For me anyway. I guess more study is needed.

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

1 participant