I just noticed that installing this version of the module on a 5.1 site resulted in the module listing itself under Core - Optional instead of Other, or Login, or whatever. I think that's a no-no? It took me 20 minutes to figure out how to install it. I was automatically closing the Modules > Core - Optional form while looking for it, because "it couldn't be in there..."

Comments

ardas’s picture

ardas commented
Assigned: Unassigned » ardas
Status: Active » Fixed

No problem with that. I was thinking about how to name it and I didn't want to create a package which would consist of this module only.

So, I would better put it into Other (just like Autoogout module or LoginToBoggan) since I don't think other variants are better. I will commit this fix in 20 mins. Please, update 5.0 release tomorrow.

Thanks.

senpai’s picture

senpai commented

I file a bug report at 2:18am. ARDAS responds within 10 minutes, and commits a patch within 30 minutes. Wow. I expected this to languish in the queue for weeks before getting a response.

This is what I call module maintenance! Nice goin ARDAS!

ardas’s picture

ardas commented

Thanks! :)

senpai’s picture

senpai commented
Status: Fixed » Closed (fixed)