One thing which makes this module hard to test and understand is that the README.txt is way too basic. Here is a start of doing it better, so that people understand what does this module do, and a little bit more on how. For example, the existing text might suggest that it requires the OpenID module turned on, but it does not. It was totally confusing for me as I read it.

CommentFileSizeAuthor
#2 litte-better-readme.patch1.16 KBalex_b
litte-better-readme.patch1.09 KBgábor hojtsy

Comments

anarcat’s picture

anarcat commented
Status: Needs review » Needs work

The URL is likely to change based on #322764: Adding patch to support new OpenID Provider Path, to user/%/identity. Otherwise ready to commit.

alex_b’s picture

alex_b commented
StatusFileSize
new litte-better-readme.patch1.16 KB

Updated to reflect changes by #322764: Adding patch to support new OpenID Provider Path.

RTBC after 322764 lands.

alex_b’s picture

alex_b commented
Status: Needs work » Needs review

Something weird is going on with these comments when I attach patches. Setting status.

anarcat’s picture

anarcat commented
Status: Needs review » Reviewed & tested by the community

Looks fine to me.

walkah’s picture

walkah commented
Status: Reviewed & tested by the community » Fixed

committed, thanks guys

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.