I just stumbled on this screen when I had no sites on there. It just said "No sites yet." It did not say how to get sites there, or what those sites are gonna be, or... It did not tell me anything. Looked like a proof of concept.

And then when I actually used it with one site, I assumed it is gonna change. It did! now it does not tell me what this checkbox means and what if I uncheck it. I cannot remove stuff, but only disable.

I believe this would be pretty important to fix up before the 1.0 release, but is also critical to make some sense before this can ever be installed on the live drupal.org site. I'd love to help out cruft the right message, but I might not be the best one to come up with the initial one. For example, would it make sense to be able to delete items from here? So that if for example you just test stuff out, it is not gonna litter your list of sites? Or should we just keep to the setting for automated assignments?

CommentFileSizeAuthor
openidsites.jpg143 KBgábor hojtsy

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Issue tags: +drupal.org redesign, +drupal.org identity

Applying tags.

anarcat’s picture

anarcat commented

From what I understand in my tests, those are the sites the openid_provider module gave access to. The checkbox says if the site is allowed "once" (unchecked) or "always" (checked). So this should probably be a table with the following legend:

| Site |       Allowed      |
|      | Always       | Ask |
| Foo  |    ( )       | (*) |
| Bar  |    (*)       | ( ) |

This is currently possible, given the database schema, as far as I understand. Now, I think that it should look like this:

Access control

| Site | Always | Ask | Deny |
| Foo  |  ( )   | (*) |  ( ) |
| Bar  |  (*)   | ( ) |  ( ) |

I.e. that we should allow users do explicitely deny access for some openid clients.

Chris Johnson’s picture

Chris Johnson commented

See this issue: http://drupal.org/node/396508

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Closed (duplicate)

Ok, making this a duplicate of that one.