CommentFileSizeAuthor
#16 2605470-16.patch52.51 KBxavip
#15 2605470-15.patch51.95 KBxavip
#14 2605470-14.patch48.96 KBxavip
#13 2605470-13.patch39.01 KBxavip
#12 2605470-12.patch38.13 KBxavip
#11 2605470-11.patch33.23 KBxavip
#10 2605470-10.patch30.12 KBxavip
#9 2605470-9.patch26.44 KBxavip
#8 2605470-8.patch17.79 KBxavip
#6 2605470-6.patch17.79 KBxavip
#4 2605470-4.patch6.02 KBxavip
#2 2605470-2.patch17.79 KBxavip

Comments

XaviP created an issue. See original summary.

xavip’s picture

xavip commented
Status: Active » Needs review
StatusFileSize
new 2605470-2.patch17.79 KB

First patch, needs work. Checking tests.

Status: Needs review » Needs work

The last submitted patch, 2: 2605470-2.patch, failed testing.

xavip’s picture

xavip commented
Issue summary: View changes
StatusFileSize
new 2605470-4.patch6.02 KB
xavip’s picture

xavip commented
Status: Needs work » Needs review
xavip’s picture

xavip commented
Issue summary: View changes
StatusFileSize
new 2605470-6.patch17.79 KB

Patch #2 corrected. Checking tests.

Status: Needs review » Needs work

The last submitted patch, 6: 2605470-6.patch, failed testing.

xavip’s picture

xavip commented
Status: Needs work » Needs review
StatusFileSize
new 2605470-8.patch17.79 KB
xavip’s picture

xavip commented
Related issues: +#2605472: Alerts for offers/demands in the network (O2), +#2622130: Email gateway
StatusFileSize
new 2605470-9.patch26.44 KB

- All templates in en, es and ca, ready for fresh installation or update.
- New db table for alerts, ready for fresh installation or update.
Needs work in coding new alerts and new function to send emails. Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-10.patch30.12 KB

Correcting templates and new db table. Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-11.patch33.23 KB

- Corrected problems updating templates in integralces.net
- Alerts admin form ready.
- Needs work in hook_cron, ces_message function to send emails and disabling alerts in user profile.
Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-12.patch38.13 KB

Implemented alerts expired and newoffer in cron.
Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-13.patch39.01 KB

- Changing default behaviour on newoffer and newwant alerts.
- Better rendering tokens.
Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-14.patch48.96 KB

Configuration of email alerts in user profile. Needs work in cron. Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-15.patch51.95 KB

All email alerts in cron. Checking tests.

xavip’s picture

xavip commented
StatusFileSize
new 2605470-16.patch52.51 KB

Cleaning code. Ready to commit.

  • XaviP committed c03ace7 on 7.x-1.x 
    Issue #2605470: All email alerts (O1, O2)
xavip’s picture

xavip commented
Status: Needs review » Fixed
xavip’s picture

xavip commented
Status: Fixed » Needs work

Needs work:
- Match periodicity in ces_alerts_form_submit() and ces_offerswants_cron() for periodic alerts.
- Render template and tokens only once for all emails of an alert, if possible.
- Send emails in series of 50 with cron queue, with a time wait of 1 minute (enough?)

  • XaviP committed 72384da on 7.x-1.x 
    Issue #2605470: Mailing in series of 50 users to queue cron each 60...
xavip’s picture

xavip commented
Status: Needs work » Fixed

Changes done.
Not possible render template and tokens only once since each user can have different prefered language.

Status: Fixed » Closed (fixed)

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