aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet/SendReport
Commit message (Collapse)AuthorAgeLines
* [SeeSomething] Remove cobrand.Matthew Somerville2017-03-31-8/+1
|
* Use sender in From if From and To domains match.Matthew Somerville2017-03-14-1/+3
| | | | | | To deal with a recipient mail server not allowing inbound email using the same domain as an internal domain, e.g. https://community.mimecast.com/docs/DOC-1419
* Remove duplicate <> around envelope senders.Matthew Somerville2017-03-14-4/+1
|
* Refactor SendReport::Open311 to use cobrand hooks.Matthew Somerville2017-03-09-70/+14
| | | | | | | | | | | | | | Avoid doing cobrand-specific logic within SendReport::Open311. The hooks defined are: open311_config: set extra fields open311_pre_send: set endpoints and munge data These are tested in the new t/app/sendreport/open311.t, which does not send any actual reports, but just checks that all the data is munged into the expected format. (tests under t/open311* should already test the actual sending) Remove unused Open311 special cases.
* Add cobrand call_hook fn, calls a fn if defined.Matthew Somerville2017-02-15-1/+1
|
* Add a 'closest_address' hidden Open311 attribute.Matthew Somerville2017-02-03-0/+2
| | | | This joins easting, northing and an external ID field.
* Move staging flags to their own config variable.Matthew Somerville2017-01-10-1/+1
|
* Automatically spot Open311 co-ord/ID attributes.Matthew Somerville2016-08-24-39/+28
|
* Pass back Open311 request through send_reports for tests.Matthew Somerville2016-08-24-0/+7
|
* Add HTML email templates.Matthew Somerville2016-08-01-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | Design is all Zarino. This adds the ability to send HTML emails, including attached inline images. When included, this is done as a multipart/related email containing a multipart/alternative (of plain and HTML) and any attached images, so that the images are available even if HTML mail is not. The alert emails list data has been improved so it can be constructed in the templates rather than the code. Various templates have been tidied. Various workarounds for email clients have been made, including: * <th> is used so that the Android 4.x mail client can give them `block` styling in the small screen media query. * Font settings defined on every table cell (<th>) so that sans-serif fonts are used in Outlook, rather than Times New Roman. * A three-column wrapper table to create a 620px centred content area that also shrinks down on narrow screens. (Outlook doesn’t like max-width, so this is the simplest alternative.) * Enforcing a sensible (500px) min-width for the main content area, on clients that don’t support media queries (eg: native Gmail app). * Giant borders on buttons so Outlook displays them * Image alignment with align rather than float.
* Use only one templating system for emails.Matthew Somerville2016-05-25-14/+5
| | | | | | | Historically, emails sent offline (alerts, questionnaires, etc) used a different templating system from those sent by the website (e.g. login emails), though the newer system was also being used for the site name and signature of offline emails.
* Store Open311 error in report on failure.Matthew Somerville2016-04-27-3/+2
| | | | | | It will still output via cron on the first failure. However, reduce the verbosity of a couple of SendReport modules which are reporting failure every time.
* [UK] Remove West Berkshire temporary fix.Matthew Somerville2016-04-27-4/+0
|
* Remove some unused cobrands.Matthew Somerville2016-04-20-58/+2
|
* [Angus] Remove two unnecessary lines that caused Encode warningsDave Arter2016-03-29-2/+0
|
* [Angus] SOAP interface/send method for Angus CRMDave Arter2016-03-29-2/+174
|
* Better handle replies to bounce addresses.Matthew Somerville2016-02-23-8/+7
| | | | | | | | | | Auto unsubscribe alert bounces, forward on report bounces and alert replies to support, and send through to report creator non-bounce replies to their report (for systems that ignore both the From and Reply-To headers). Also forward any totally unparsed bounce to support to possibly then adjust this bounce handling.
* Remove many uses of mySociety::Config.Matthew Somerville2015-12-02-8/+7
|
* Move from Moose to Moo in non-App code.Matthew Somerville2015-12-02-7/+7
| | | | And create default cobrand class without all of Moose.
* Reduce use of FixMyStreet::App.Matthew Somerville2015-12-02-10/+7
| | | | Command line scripts don't need a full blown app, just database.
* Move EmailSend out of Catalyst for use elsewhere.Matthew Somerville2015-12-02-1/+2
|
* [Zurich] Updated email templatesMatthew Somerville2015-10-06-0/+10
| | | | | | | - submit-external-wish.txt includes a link with report's coordinates, template values are interpolated Conflicts: perllib/FixMyStreet/SendReport/Zurich.pm
* [Zurich] admin workflow changes available statesHakim Cassimally2015-10-06-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | See mysociety/FixMyStreet-Commercial#690 - external_message for Extern/Wunsch states - as per PDF mockups, the "public response" field is now editable from more states, namely: - Ruckmeldung ausstehend - Extern - Zustandigkeit unbekannt - Wunsch - Nicht kontaktierbar - Unsichtbar - test fixes NB: slight hack to reorder Zurich report_display tests As these remove 'investigating' from hidden_states, these break tests for default cobrand (the FMS::App code expects to be called once on request start, not multiple times within one test) - Not contactable button and tests for this and the similar `send_back` functionality (previously untested). NB: Unsure why we need to specify form_number=2 just for these submits, but test won't find the buttons otherwise. - Make sure SDMs' "time_spent" figure is recorded. (See mysociety/FixMyStreet-Commercial#718)
* Allow attachment of emails in email_sendHakim Cassimally2015-10-06-1/+6
| | | | Required by Zurich for mysociety/FixMyStreet-Commercial#675
* [fixmystreet.com] Remove unused Barnet code.Matthew Somerville2015-09-03-208/+0
|
* Test for DMARC when sending report email.Matthew Somerville2015-08-19-0/+7
|
* Merge branch 'issues/968-refusing-councils'Matthew Somerville2015-07-28-0/+7
|\
| * [fixmystreet.com] Unresponsive bodies page/warningMatthew Somerville2015-07-28-0/+7
| | | | | | | | | | | | | | | | | | Add a special category email address used to show an unresponsive message and page when trying to report in that category. Add a "Refused" send method for setting a whole body to be unresponsive and show the message immediately, not on category selection. Factor out category template, and put at top where needed.
* | [fixmystreet.com] Remove Essex/Oxfordshire code.Matthew Somerville2015-07-22-27/+0
| | | | | | | | This is no longer used.
* | Set external_id for Greenwich Open311 based on bodies_str, not cobrandDave Arter2015-07-20-1/+2
|/ | | | | Problems for Greenwich reported via the fixmystreet cobrand weren't having the external_id included in the Open311 request and were failing.
* [Greenwich] Include external_id when submitting Open311 reportsDave Arter2015-06-30-2/+2
|
* Add Extra role to ease use of {extra} field.Hakim Cassimally2015-03-20-10/+8
| | | | | | | | | Historically, the extra field has been used in two different ways by different cobrands, both as a list (e.g. Open311 category fields) and a hash (e.g. the Zurich cobrand). This commit consolidates usage, adding an API to make use of the field easier and always returning a hash for the code to use. Fixes #1018.
* [fixmystreet.com] Remove special NI code.Matthew Somerville2015-03-16-37/+0
| | | | | This is now handled by normal bodies in the admin. In order to remove references in send_reports, tidy up (unused) EmptyHomes code a bit.
* [fixmystreet.com] Remove special Report-It code.Matthew Somerville2015-03-13-113/+0
|
* [Oxfordshire] Update email template for cobrand.Matthew Somerville2015-01-21-1/+4
| | | | And update HTTPS link.
* Remove need for cron-wrapper with existing scriptsMatthew Somerville2015-01-19-0/+1
| | | | | Call the necessary boilerplate in each script so you can call them directly. Remove boilerplate from files that don't need it.
* Use same handling for cron and non-cron email.Matthew Somerville2015-01-13-29/+21
| | | | | This means that e.g. SMTP authentication is used when set up by all emails, not just non-cron ones. Fixes #988.
* Pass cobrand into send_email_cron when sending SendReport emailsSteven Day2014-12-09-1/+3
| | | | | | | | | | | | | | Without this, email sending within a cobrand will get the right main email template (submit.txt) for the email - because it uses the cobrand inside get_template - but it doesn't use the cobrand for things like the email signature, because you have to pass it into send_email_cron manually to make it cobrand-aware. This probably hasn't been noticed before because most cobrands are called Something FixMyStreet, but Collideoscope is different and hence it caused a problem there. Closes mysociety/FixMyStreet-Commercial#621.
* Remove liberror-perl and libsoap-lite-perl.Matthew Somerville2014-11-05-5/+5
| | | | | | Try and save other people the potential pain of installing SOAP::Lite; only UK FixMyStreet and Zurich geocoder use it, so set it as an optional feature in the cpanfile instead.
* Open311 tweaks to get round-tripping workingHakim Cassimally2014-10-16-2/+8
| | | | | If https://github.com/mysociety/fixmystreet/pull/792 is accepted then this should be rewritten in terms of that.
* refactor repeat code into get_email_templateHakim Cassimally2014-08-14-6/+2
|
* Add no-op sending method.Matthew Somerville2014-03-22-0/+12
| | | | | This simply always skips reports for this body, in case you need to suspend reports there. Fixes #507.
* Change default email indentation to none.Matthew Somerville2014-03-17-1/+0
| | | | | This is very old, no-one seems to want/need it any more, and more than one reuser does not want it. Fixes #715.
* Send staging reports to reporter, not contact address.Matthew Somerville2014-01-30-1/+1
| | | | Fixes #653.
* change email sender to handle multiple emails per contactStruan Donald2013-10-28-3/+13
| | | | | splits emails on comma WIP
* Use name from problem in Bromley Open311 extra paramsStruan Donald2013-08-19-1/+1
| | | | | In order to match the way we get the name for the standard Open311 params as changed in b3e8d60
* Fixes to module installation.Matthew Somerville2013-07-26-1/+1
| | | | | | | * Upgrade ExtUtils::MakeMaker, and remove old workaround. * DIME::Tools won't install without --force (needed by SOAP::Lite in travis). * Remove SOAP::WSDL from cpanfile, as it won't install and nothing needs it apart from Barnet (UK specific).
* Consolidate sending backoff for different types of sender.Matthew Somerville2013-05-02-45/+0
|
* Don't print on London failure every time.Matthew Somerville2013-04-29-1/+0
|
* Merge branch 'zurich'Matthew Somerville2013-03-14-1/+14
|\ | | | | | | | | | | | | | | | | Conflicts: locale/FixMyStreet.po locale/cy_GB.UTF-8/LC_MESSAGES/EmptyHomes.po locale/de_CH.UTF-8/LC_MESSAGES/FixMyStreet.po locale/nb_NO.UTF-8/LC_MESSAGES/FixMyStreet.po locale/nn_NO.UTF-8/LC_MESSAGES/FixMyStreet.po