Skip to content
This repository has been archived by the owner on Aug 23, 2021. It is now read-only.

use cloud v2 endpoints, remove orgId #285

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Conversation

jamespohalloran
Copy link
Contributor

Two new endpoints that the client interfaces with have changed along with the Cognito shared-userpool project:

  • [identiyAPI]/v2/apps/[appId]/currentUser
  • [contentAPI]/content/[appId]/github/main

The orgId is no longer needed within the client, so the bulk of this change is removing orgId from everywhere it's used/referenced.

@changeset-bot
Copy link

changeset-bot bot commented Jun 23, 2021

🦋 Changeset detected

Latest commit: 976175e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
demo Minor
tina-tailwind-sidebar-demo Minor
next-tinacms-cloudinary Major
tina-cloud-next Minor
tina-graphql-gateway Minor
tina-cloud-starter-experimental Patch
tina-cloud-starter Patch
test Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jamespohalloran jamespohalloran temporarily deployed to test June 23, 2021 16:36 Inactive
@jamespohalloran
Copy link
Contributor Author

Would the easiest way to test this be to create a canary release? @jeffsee55 (I just don't want users to accidentally install this version until we have exposed the new v2 dashboard).

@jeffsee55
Copy link
Member

Yep that makes sense to me @jamespohalloran, you can just release those from local - I can walk you through it if you haven't done it before.

@mittonface mittonface temporarily deployed to test July 9, 2021 13:18 Inactive
@mittonface mittonface temporarily deployed to test July 9, 2021 13:22 Inactive
@mittonface

This comment has been minimized.

@jeffsee55
Copy link
Member

@mittonface, just updated. Can you try again?

@mittonface
Copy link
Contributor

@mittonface, just updated. Can you try again?

Looks good now. Thanks!

@mittonface mittonface temporarily deployed to test July 13, 2021 15:59 Inactive
@mittonface mittonface temporarily deployed to test July 13, 2021 18:11 Inactive
@mittonface mittonface temporarily deployed to test July 13, 2021 18:41 Inactive
@logan-anderson
Copy link
Contributor

@jeffsee55 @mittonface What is the work that is left to do here besides making sure it is up to date with the latest version in master?

@mittonface
Copy link
Contributor

Nothing left to do here. Just want to be clear that this is a breaking change:

Before this is merged:

  • You get your own unique userpool
  • You'll be calling the v1 versions of apis
  • you access your dashboard at https://orgname.tina.io

After this is merged:

  • You connect to the SaaS userpool
  • You'll be calling the v2 version of the apis
  • you access your dashboard at https://app.tina.io

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants