Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

Added content on Pantheon-extended privacy #1771

Merged
merged 36 commits into from
Jul 29, 2019

Conversation

MadelineMurray
Copy link
Contributor

Added content on Pantheon-extended privacy including:

  • Pantheon-extended privacy
  • How to access private and privacy marker transactions
  • How to create and manage privacy groups


# Using Pantheon-extended Privacy

Pantheon provides an extended implementation of privacy. When using Pantheon-extended privacy, a [privacy
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor suggestion:
Pantheon provides an extended privacy implementation by allowing a privacy group to be created for a set of participants with a unique privacy group ID. The privacy group ID must be specified when sending private transactions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

to create an EEA-compliant private transaction.

!!! note
Support for specifying `privacyGroupId` when using eea_sendTransaction with EthSigner will be available in
Copy link
Contributor

Choose a reason for hiding this comment

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

must eea_sendTransaction be formatted?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

Minor suggestions only

@MadelineMurray MadelineMurray merged commit a23b011 into PegaSysEng:master Jul 29, 2019
@MadelineMurray MadelineMurray deleted the pantheonPrivacy branch July 29, 2019 05:56
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.

2 participants