Hey again,

Sorry to be annoying, but this module is all about users and roles and registration, and all those settings are located in admin/user, not admin/settings, so it would be very nice of you if you could move admin/settings/registration_role_with_approval to admin/user/registration_role_with_approval

cheers :)

CommentFileSizeAuthor
#4 rrwa_cleanup_v3.patch8.71 KBarski
#3 rrwa_cleanup_v2.patch8.7 KBarski
#1 rrwa_cleanup.patch8.59 KBarski

Comments

arski’s picture

arski commented
StatusFileSize
new rrwa_cleanup.patch8.59 KB

Hey, submitting a patch with a full cleanup of the module. No functionality changed, just made the code look appropriately :/

arski’s picture

arski commented
Status: Active » Needs review
arski’s picture

arski commented
StatusFileSize
new rrwa_cleanup_v2.patch8.7 KB

missed some :/

arski’s picture

arski commented
StatusFileSize
new rrwa_cleanup_v3.patch8.71 KB

and removed too much.. now it should be right.

toni04’s picture

toni04 commented
Issue summary: View changes
Status: Needs review » Fixed

Patch reviewed and working.

tamerzg’s picture

tamerzg commented
Status: Fixed » Patch (to be ported)
tamerzg’s picture

tamerzg commented
Status: Patch (to be ported) » Fixed

Patch applied.

Status: Fixed » Closed (fixed)

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