Hi,

I've just installed this module and am testing it. Looks very promising for my purposes. I hope to post issues as I find them, using a separate post for each distinct issue so you can track them more accurately (I hope!). Thanks for the module.

First issue is the webadmin DB field is only 72 characters. Mailmain URLs can easily exceed this, as I found in my first test. I'm not sure what a reasonable maximum length might be, but 72 is way too short!

- Thanks

Comments

cvining’s picture

cvining commented
cvining’s picture

cvining commented
Title: URL field in DB table mailman_lists too short » Mailing Lists Tabs under User Account
Component: Code » User interface
Category: bug » feature

There are two tabs under "User account"-->Edit
1. Mailing Lists
2. Mailing Lists Register

They seem a bit redundant and I'd rather hide the first tab and use only the second. It is simpler for the user, I think.

Also, I rather see Mailing Lists Register promoted one tab level, so you see it directly under
"User account"-->Mailing Lists Register

Mailing lists are a high priority and I'd rather the user find those settings right away.

Thanks for your consideration!

cvining’s picture

cvining commented
Title: Mailing Lists Tabs under User Account » Proper sync when user changes email address
Component: User interface » Code

This is the main feature I need in a maillist manager today. Users do change their email address from time to time. They need a simple way to update one thing (their drupal account) and have the mail lists automatically updated too.

The code to do this may not be too difficult with the present module. The main thing is to detect when a user changes their drupal email address, check if it is subscribed to a list (or lists), then unsubscribe the old one and subscribe the new one.

The present code will loose sync altogether when a user changes their drupal email address.

Thanks again for the nice module.

- Cronin

samuelet’s picture

samuelet commented

Thanks for your feedback.
Webadmin field lenght is now fixed with varchar 255.

About tabs, there is a quick tutorial at the end of my project page.
After you have denied mailman manager to your users, you'd need to apply the patch to correctly hide the empty form.
Pay attention that administrator, but only himself, should still display the tab just because he has already a mailman manager access permission.

About profile sync, i'm going to fix it.

cvining’s picture

cvining commented

Excellent! With those changes, I'll be quite the happy camper!

- Cronin

samuelet’s picture

samuelet commented
Status: Active » Fixed

It should be fixed in with the #90118 patch.
Until i'll not implement a pending queue for missed subscriscription request, it could happen a lost of sync when user change his email address and mailman server is unavaiable.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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