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

moving UCR from auth panel #41

Merged
merged 2 commits into from
May 29, 2020
Merged

moving UCR from auth panel #41

merged 2 commits into from
May 29, 2020

Conversation

elf-pavlik
Copy link
Member

@elf-pavlik elf-pavlik commented May 18, 2020

This suppose to provide starting point for Use Cases and Requirements document. Current content comes directly from https://github.com/solid/authorization-and-access-control-panel/pull/66/files

EDIT rendered HTML preview

@csarven
Copy link
Member

csarven commented May 24, 2020

Great kick off! I suggest changing the order; Use Cases normally precede Requirements.

If you can connect the UCs to the User Stories at https://github.com/solid/user-stories (see issues), that'd be very helpful. We also need some kind of feedback from the community (survey).

See the first comment in solid/specification#9 for general direction and also links to example UCRs from W3C TR.

@justinwb
Copy link
Member

justinwb commented May 26, 2020

@elf-pavlik I had a discussion with @csarven on friday about putting these directly into the solid/specification repo. For myself I think this is a good idea. I think we can tune it up a bit in here (get the structure right, etc), and then submit it into the specification repo as a proposal. IMO we should try to get that done quickly in here, and then operate on it directly under the solid/specification repo moving forward

use-cases-and-requirements.bs Outdated Show resolved Hide resolved
Introduction {#intro}
=====================

Issue: Needs explaination how proposals should clearly reference requirements which they satisfy. As well as requirements they do not satisfy due to some limitations.
Copy link
Member

Choose a reason for hiding this comment

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

I think it's also important to provide some expectations around use case scope. We should focus mostly on lean and easy to understand use cases and requirement that don't drift too far away from practical use, so they can be as informative and referenceable as possible.


Requirements {#requirements}
============================

Copy link
Member

Choose a reason for hiding this comment

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

I feel like requirements should be categorized (e.g. authentication, authorization, identity, validation, resource access, security), but use cases should specifically not be, since good ones should span categories.


Use Cases {#use-cases}
======================

Copy link
Member

Choose a reason for hiding this comment

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

It feels like the use cases should be organized more based on the unique pattern that they're demonstrating, so they can be found and navigated more intuitively.

@elf-pavlik
Copy link
Member Author

Instead of trying to address all the ideas in this PR I would prefer to merge it and invite everyone to submit following up PRs with amendments. In case anyone sees problem but doesn't feel ready to submit PR we can open an issue and have preliminary discussion there.

I think we can tune it up a bit in here (get the structure right, etc), and then submit it into the specification repo as a proposal.

SGTM let's iterate via multiple PRs

@csarven
Copy link
Member

csarven commented May 28, 2020

I've no objection to moving any changes to a different PR but I'm not sure if a review here serves much purpose. Go for it I guess.

@justinwb
Copy link
Member

Merging this in - will iterate in successive PRs to incorporate some of the feedback left in this one.

@justinwb justinwb merged commit 00315b4 into solid:master May 29, 2020
@elf-pavlik elf-pavlik deleted the ucr branch June 17, 2020 12:53
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.

3 participants