Skip to content

Conversation

@DyrellC
Copy link
Owner

@DyrellC DyrellC commented Jul 12, 2022

Description of change

Currently the only place we use Permissioned is when we are doing checks on Keyload messages, but those permissions are not retained in store anywhere, as our cursor store only uses the Identifier. To allow for that permission retention we need to change CursorStore to map by Permissioned<Identifier> instead of just Identifier. This brings a host of other trickling changes along with it.

For example, a User will be able to check their permissions on a topic before they attempt to send messages, failing if they are registered as Read by a topic administrator. Additionally administrative privileges can be properly assigned and forwarded through the topics.

Type of change

  • Enhancement (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How the change has been tested

New section added to end of examples to have Author re-subscribe SubscriberA and grant them Admin privileges, confirming that they carry forward into new branches.

  • All previous examples complete
  • All previous tests complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants