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

Additions to custom user data #6221

Merged

Conversation

elle-j
Copy link
Member

@elle-j elle-j commented Oct 30, 2023

What, How & Why?

Some additions/updates to the custom user data addition to the example app:

  • Adds permission role to the custom data collection to only read and write to your own.
  • Updates the onUserCreation callback to be run as an Application Authentication in order to run in the context of the existing app user.
  • Types the user data for type safety.
  • Updates the README.

☑️ ToDos

--

@coveralls-official
Copy link

coveralls-official bot commented Oct 30, 2023

Coverage Status

coverage: 85.397%. remained the same when pulling 53afe75 on lj/custom-user-data-additions into 14490d2 on andrew/custom-user-data-example.

Comment on lines +87 to +91
<View style={styles.status}>
<Text style={styles.statusText}>
Team: {user.customData.team}
</Text>
</View>
Copy link
Member Author

Choose a reason for hiding this comment

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

Nit: Just moved this above the "Connected" text.

But also, removed the ? after customData as it will always return a record.

@elle-j elle-j mentioned this pull request Oct 30, 2023
7 tasks
Copy link
Member Author

Choose a reason for hiding this comment

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

The changes here and some in StoreScreen.tsx (i.e. moving the responsibility of deleting the user to this file) are the same as in the previous PR. But since this one will be merged after, the previous changes probably would have been overwritten had it not been changed here as well.

Copy link
Contributor

@takameyer takameyer left a comment

Choose a reason for hiding this comment

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

Thanks so much for putting some final polish on this!

@takameyer takameyer merged commit 3da2229 into andrew/custom-user-data-example Oct 31, 2023
27 of 29 checks passed
@takameyer takameyer deleted the lj/custom-user-data-additions branch October 31, 2023 12:57
takameyer pushed a commit that referenced this pull request Oct 31, 2023
takameyer pushed a commit that referenced this pull request Oct 31, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
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.

None yet

2 participants