Skip to content

Conversation

@justinwb
Copy link
Member

@justinwb justinwb commented Jul 31, 2020

Introduces Data Authorization workflows, with Access Needs, Access Grants, Access Receipts, and Remote Data Registration.

@dmitrizagidulin
Copy link
Member

Very interesting direction! At least on first reading, I'm finding the multi-part AccessDecorator(Index/Series/etc) system to be really hard to understand. But maybe it'll get clearer on multiple readings.

Copy link
Member

@elf-pavlik elf-pavlik left a comment

Choose a reason for hiding this comment

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

My two inline comments should explain why we may not need to define inverse counterpart for any property.

@justinwb
Copy link
Member Author

Very interesting direction! At least on first reading, I'm finding the multi-part AccessDecorator(Index/Series/etc) system to be really hard to understand. But maybe it'll get clearer on multiple readings.

Will be adding text to explain that, but essentially the decorators allow us to add labels and descriptions to data without tying them directly to it. This allows us to have access needs that can be described in different languages, or (depending on the context) different ways

@ericprud
Copy link
Contributor

Very interesting direction! At least on first reading, I'm finding the multi-part AccessDecorator(Index/Series/etc) system to be really hard to understand. But maybe it'll get clearer on multiple readings.

Will be adding text to explain that, but essentially the decorators allow us to add labels and descriptions to data without tying them directly to it. This allows us to have access needs that can be described in different languages, or (depending on the context) different ways

Maybe this would be improved by a leading example UI which associates features (text comes from the AppDecorator) with their required resources (text comes from the ShapeTreeDecorator).

(i miss email threads, quoting is a noisy substitute)

@justinwb justinwb marked this pull request as ready for review September 30, 2020 21:08
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.

7 participants