Skip to content
This repository was archived by the owner on Aug 14, 2024. It is now read-only.

expand data handling#663

Merged
bruno-garcia merged 4 commits into
masterfrom
feat/expand-data-handling
Aug 11, 2022
Merged

expand data handling#663
bruno-garcia merged 4 commits into
masterfrom
feat/expand-data-handling

Conversation

@bruno-garcia
Copy link
Copy Markdown
Member

@bruno-garcia bruno-garcia commented Aug 7, 2022

  • Being more explicit about our SDK guidelines for handling sensitive data
  • Point out that changes to Relay have to happen to influence server side PII stripping
  • Add to Philosophy
  • Reorder pages to become a more sequential flow for learning

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
develop ✅ Ready (Inspect) Visit Preview Aug 11, 2022 at 3:46PM (UTC)

Copy link
Copy Markdown
Member

@dcramer dcramer left a comment

Choose a reason for hiding this comment

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

prob some general language cleanup that could happen here too - maybe @imatwawana can help there

Comment thread src/docs/sdk/data-handling.mdx Outdated
Comment thread src/docs/sdk/data-handling.mdx Outdated
Comment thread src/docs/sdk/data-handling.mdx Outdated
Comment thread src/docs/sdk/data-handling.mdx Outdated
Comment thread src/docs/sdk/philosophy.mdx Outdated
@bruno-garcia
Copy link
Copy Markdown
Member Author

Applied the updates, appreciate another stab at reviews please

Copy link
Copy Markdown
Contributor

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

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

I think this is a good start. Do we want to add a note about PII vs. potentially sensitive data?

@bruno-garcia
Copy link
Copy Markdown
Member Author

I think this is a good start. Do we want to add a note about PII vs. potentially sensitive data?

Feel free to open a follow up, lets merge this for now as it improves things a bit

@bruno-garcia bruno-garcia merged commit accc5a0 into master Aug 11, 2022
@bruno-garcia bruno-garcia deleted the feat/expand-data-handling branch August 11, 2022 20:06
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.

3 participants