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

Add use case for client to discover access privileges #136

Merged
merged 5 commits into from
Dec 2, 2020

Conversation

csarven
Copy link
Member

@csarven csarven commented Nov 25, 2020

Based off a number of existing discussions and work:

Certainly there are other issues lying around... and this is a fairly common. It also goes along with some of the other use cases.

Aside: solid/specification#210

Copy link
Contributor

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

Suggested some language adjustment where I think I could understand what was intended, and requested some more where I could not understand what was intended.

proposals/wac-ucr/index.bs Outdated Show resolved Hide resolved
proposals/wac-ucr/index.bs Outdated Show resolved Hide resolved
proposals/wac-ucr/index.bs Outdated Show resolved Hide resolved
Co-authored-by: Ted Thibodeau Jr <[email protected]>
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.

Explains the need well 👍


For example, for user level permission, if the application is not granted
[=write access=] on the resource that Guinan is currently editing, the user
interface can disable the "Save" button in the menu. Guinan also wants to know
Copy link
Member

@elf-pavlik elf-pavlik Dec 2, 2020

Choose a reason for hiding this comment

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

I would prefer to end this use case here and move "Guinan also wants to know ..." to a dedicated use case. I can make PR after merge.

Copy link
Contributor

@bblfish bblfish left a comment

Choose a reason for hiding this comment

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

looks good :-)

@csarven csarven merged commit b0e2ab9 into master Dec 2, 2020
@matthieubosquet matthieubosquet deleted the uc-client-discover-access-privileges branch March 3, 2021 13:51
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.

5 participants