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

Public commenting for extended social network #98

Merged
merged 1 commit into from
Sep 2, 2020
Merged

Public commenting for extended social network #98

merged 1 commit into from
Sep 2, 2020

Conversation

bblfish
Copy link
Member

@bblfish bblfish commented Aug 5, 2020

Allowing people to fight spam on their blog posts without creating a filtering bottleneck.
Following on the Minimal Credentials Disclosure use case
#95 (review)

@justinwb justinwb requested a review from a team August 19, 2020 14:10
@@ -314,6 +314,16 @@ the output of their shared work. She gives him [=control access=] to
`project1` so that he can help her invite other colleagues from the past to
add [=resources=] to it.

### Public commenting for extended network ### {#collection-foaf}
Copy link
Member

Choose a reason for hiding this comment

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

  • Since this use case is based on inherited permissions it should go under the "Inheritance" section.
  • A title like "Default permissions for extended network" might be more accurate? Reading through it, it sounds like this would be a rule set at the container level that evaluates the creator of the comment to determine if they are within Alice's extended network, and then applies the permissions based on the result?

Copy link
Member

@justinwb justinwb left a comment

Choose a reason for hiding this comment

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

Action item for Justin to merge and then move to Inheritance section per comments on PR and update title

@justinwb justinwb merged commit 9aedfd3 into solid:master Sep 2, 2020
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.

4 participants