Opened 6 years ago

Closed 6 years ago

#1955 closed bug (fixed)

Limit mass mailing to voters to members logged in in the last 6 months

Reported by: shevek Owned by: shevek
Priority: major Milestone: 1.5
Component: BW Admin Keywords:
Cc: jsfan, pablobd

Description

Using the massmail tool the people allowed to vote are choosen randomly should be choosen from members logged in in the last 6 months.

Change History (12)

comment:1 Changed 6 years ago by shevek

  • Owner set to shevek
  • Status changed from new to assigned

comment:2 Changed 6 years ago by shevek

  • Cc jsfan added
  • Milestone changed from 1.6-proposed to 1.5
  • Status changed from assigned to to_alpha

comment:3 Changed 6 years ago by jsfan

  • Status changed from to_alpha to testing

comment:4 Changed 6 years ago by shevek

  • Cc pablobd added

Enqueued a mass mail to 30 members and checked some. All are active members and logged in during the last 6 months. I suggest to close.

comment:5 Changed 6 years ago by jsfan

  • Resolution set to fixed
  • Status changed from testing to closed

As this can't easily be tested on a local install and not many volunteers would be able to test on alpha, I'll close it based on pablobd's tests.

comment:6 Changed 6 years ago by shevek

Pablo probably didn't test yet :)

You could login some of the members in the local DB and check the enqueued list. Only right needed is massmail which should be active for the admin in the test db.

comment:7 Changed 6 years ago by jsfan

  • Resolution fixed deleted
  • Status changed from closed to reopened

Oops, sorry. I got the messages mixed up and thought pablobd had replied here as well.

Re-opening for now.

This might be unrelated but the count I get displayed is always the actual count +1. Also, I get plenty of warnings.

However, no retired profiles seem to be selected at least. All looks good as far as this ticket is concerned.

comment:8 Changed 6 years ago by shevek

I enqueued on alpha as Pablo gave me the rights. Numbers where right there. If the newsletter was already enqueued to someone the numbers add up.

Which warnings are shown? Alpha doesn't show any.

comment:9 Changed 6 years ago by shevek

  • Status changed from reopened to to_alpha

Fix is deployed on alpha. Changing status accordingly.

comment:10 Changed 6 years ago by shevek

  • Status changed from to_alpha to testing

comment:11 Changed 6 years ago by pablobd

Looks fine, enqueued all active users

comment:12 Changed 6 years ago by shevek

  • Resolution set to fixed
  • Status changed from testing to closed

Great. Closing as fixed then.

Note: See TracTickets for help on using tickets.