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

Switch to offline session tokens #908

Conversation

mkevinosullivan
Copy link
Contributor

WHY are these changes introduced?

Current guidance is to prefer offline to online session tokens

Closes https://github.com/Shopify/first-party-library-planning/issues/355

WHAT is this pull request doing?

  • Change any reference to online from true to false
  • Remove GraphQL proxy capability (requires online session tokens)
  • Change /api/products-count to /api/products/count (aligned with change in frontend)
  • Add /api/products/create endpoint for frontend (FE previously used GraphQL proxy)

@mkevinosullivan mkevinosullivan force-pushed the kos/switch_auth_to_offline_tokens branch 2 times, most recently from 302b755 to 12d7722 Compare June 24, 2022 20:59
@mkevinosullivan mkevinosullivan changed the base branch from cli_three to cli_three_feature_v1_0_0_next June 30, 2022 21:24
@paulomarg paulomarg force-pushed the kos/switch_auth_to_offline_tokens branch from 12d7722 to c68ed93 Compare July 6, 2022 17:36
@paulomarg paulomarg self-requested a review July 6, 2022 17:36
@paulomarg paulomarg force-pushed the kos/switch_auth_to_offline_tokens branch 2 times, most recently from 6eec080 to 901317f Compare July 6, 2022 18:08
@paulomarg paulomarg force-pushed the kos/switch_auth_to_offline_tokens branch from 901317f to 34ed95f Compare July 7, 2022 15:40
Copy link
Collaborator

@paulomarg paulomarg left a comment

Choose a reason for hiding this comment

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

Just nits, but happy with this once they're dealt with.

README.md Show resolved Hide resolved
web/index.js Show resolved Hide resolved
@mkevinosullivan mkevinosullivan merged commit 6222da0 into cli_three_feature_v1_0_0_next Jul 8, 2022
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