Code review comment for lp:~xaav/wikkid/auth-provider

Revision history for this message
Tim Penhey (thumper) wrote :

On Wed, 15 Jun 2011 13:31:04 you wrote:
> > How do you feel this would be used?
> Basically wikkid would just verify whether an action was allowed or not,
> and if not display an access denied page.
> What about some general roles? Like View and Edit? I was considering the
> > possibility of having "locked" pages, using the first line of of the wiki
> > page, a little like he way moin does it.
> The way it is now is that it is up to the authorization provider to manage
> roles, allowing the most flexibility.

Well... we probably don't want that entirely.

If you think about what we want people to do, there isn't really much.

Pretty much just View and Edit.

By passing through the View name, we are expecting all providers to handle all
views, and by adding a view this would require a change to all providers.

We don't want that.

« Back to merge proposal