Update the Session to return all user information - Githubissues

Shopify / shopify-api-js

Shopify Admin API Library for Node. Accelerate development with support for authentication, graphql proxy, webhooks

MIT License

941 stars 393 forks source link

Update the Session to return all user information #1288

Closed lizkenyon closed 5 months ago

lizkenyon commented 6 months ago

WHY are these changes introduced?

First PR to start allowing saving of the Session user information. (find current user)
Saving the user information will allow developers to introduce roll based access control for pages of their app, or log user activity.

WHAT is this pull request doing?

Update the toPropertyArray, and fromProperty array to handle the additional user information that was already on the Session object we got during the OAuth
Once all of the Session Adapters have been updated we will deprecate the prior way of handling data that removes the additional user information. Currently both the new and old formats should both work in tandem.

Type of change

[ ] Patch: Bug (non-breaking change which fixes an issue)
[x] Minor: New feature (non-breaking change which adds functionality)
[ ] Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] I have used yarn changeset to create a draft changelog entry (do NOT update the CHANGELOG.md file manually)
[x] I have added/updated tests for this change
[x] I have documented new APIs/updated the documentation for modified APIs (for public APIs)