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

Updated @shopify/shopify-app-session-storage-prisma's README with latest Session model schema #1982

Merged
merged 3 commits into from
Jan 20, 2025

Conversation

mmestiyak
Copy link
Contributor

WHY are these changes introduced?

Fixes #0000

The Session model schema in the documentation was outdated and did not reflect the latest structure in the schema.prisma file. This update ensures developers have accurate information when implementing session storage with Prisma.

WHAT is this pull request doing?

This PR updates the Session model schema in the README to match the latest version. The updated schema includes all required fields and optional fields for accurate representation.

Updated schema:

model Session {
  id            String    @id
  shop          String
  state         String
  isOnline      Boolean   @default(false)
  scope         String?
  expires       DateTime?
  accessToken   String
  userId        BigInt?
  firstName     String?
  lastName      String?
  email         String?
  accountOwner  Boolean?
  locale        String?
  collaborator  Boolean?
  emailVerified Boolean?
}


## Type of change

- [ ] Patch: Bug (non-breaking change which fixes an issue)


@mmestiyak mmestiyak requested a review from a team as a code owner January 17, 2025 13:35
@mmestiyak
Copy link
Contributor Author

I have signed the CLA!

@mmestiyak mmestiyak changed the title Updated README with latest Session model schema Updated @shopify/shopify-app-session-storage-prisma's README with latest Session model schema Jan 17, 2025
Copy link
Contributor

@Arkham Arkham left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@Arkham Arkham enabled auto-merge January 20, 2025 11:49
@Arkham Arkham disabled auto-merge January 20, 2025 11:51
@Arkham Arkham merged commit adf4bd4 into Shopify:main Jan 20, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants