| Commit message (Collapse) | Author | Age | Lines |
... | |
|/ |
|
|
|
|
|
| |
This now lets the response priority and defect type handling share the
same code.
|
| |
|
|
|
|
| |
This means that the tests can be run in parallel.
|
|
|
|
|
|
|
|
|
|
|
|
| |
Historically in UK cobrands, bodies have had IDs the same as the MapIt
area ID they cover. This can be confusing (if you are setting up a dev
environment, say) and should not be necessary. This commit removes the
requirement entirely, by switching any ID checks to either the name of
the body, or the actual area it covers.
One note: the body name in the test has to match so that we do not get
two bodies both covering 2237 created. This will not be necessary when
the tests are compartmentalized in the next commit.
|
| |
|
| |
|
|\ |
|
| |
| |
| |
| |
| | |
It is possible the report's state isn't one that the user can access,
but in that case we should still show it to avoid confusion.
|
| |
| |
| |
| |
| | |
Without the report_instruct permission, the 'action scheduled'
state is not an option in the dropdown.
|
| |
| |
| |
| | |
This lets it look the same as admin/inspect uses, with optgroups.
|
|/
|
|
|
| |
Switch it to use our mock MapIt. Also remove other unused
overrides to external services.
|
|
|
|
|
|
| |
If the creation of this test report and the loading of the dashboard
took place within the same second, the test report would appear in a
different column than the test expected.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
The report-sending code has allowed multiple emails for a long time, but the
admin wouldn't allow you to enter multiple emails for each contact.
Fixes mysociety/FixMyStreet-Commercial#835
|
| |
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Email::Send is long deprecated and uses submodules that no longer work
correctly (e.g. Net::SMTP::TLS breaks with recent IO::Socket::SSL). We
create an Email::Sender subclass to perform the same functionality and
this also simplifies the email code with simpler envelope handling.
Bundle Email::Sender::Transport::SMTP to include fix from
https://github.com/rjbs/Email-Sender/issues/46
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
The code was adding two days and seven/eight hours to a floating
DateTime, which from 5-7pm on 23rd March 2017 created a DateTime
between 1-2am on 26th March which does not exist in e.g. the UK.
|
| | | |
|
| |/ |
|
|/
|
|
|
|
|
|
|
| |
Problems can have an associated defect type, that can be assigned during
an inspection. Include an admin interface for managing these types, that
can also be assigned on a per-category basis, currently available to the
Oxfordshire cobrand.
(Also include 'TM' in traffic management Exor RDI output.)
|
|
|
|
| |
This has been running for a year, added in e17eb20.
|
|\ |
|
| |
| |
| |
| | |
This works better if e.g. a user stops being associated with a body.
|
|/
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Fixes mysociety/fixmystreetforcouncils#98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rather than assigning +1 to a user's reputation every time a report is
inspected, this allows cobrands to specify which states should increment or
decrement the reputation value when a report is inspected.
The default behaviour is for reputation to never be changed, but the Oxfordshire
cobrand will increment the reputation when a report is marked as 'action
scheduled'.
For mysociety/fixmystreetforcouncils#119
|
|
|
|
|
|
| |
A report is now considered 'inspected' if its state was changed by an inspector.
Additionally, an AdminLog entry is created so the time of inspection and
inspector can be recorded.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This script can be used for the bulk closure of old reports that haven't been
updated in a long time.
Reports that haven't been updated in a *really* long time (since 2014, by
default) are simply closed, and those that haven't been updated since 2015 (by
default) are closed and an email explaining the closure is sent to the report
creator.
This script can be used for other cobrands, but they'll need an
archive.{txt,html} template in order for the emails to be sent.
For mysociety/fixmystreetforcouncils#51
|
|
|
|
|
|
| |
This method creates a token that logs the reporting user in, optionally
with some parameters which are stored with the token for use when redirecting
after login.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Limits the state dropdown to those states which will be useful to OCC
inspectors, as well as removing a few from the base template on the inspect
form. Some slight template refactoring has made it easier for other cobrands to
override the list, as well as sharing the template with the admin report_edit
form to allow all states to be selected.
Fixes mysociety/fixmystreetforcouncils#129
|
|
|
|
| |
Fixes mysociety/fixmystreetforcouncils#159
|
| |
|
| |
|
|
|
|
| |
Bromley had this already in its update state change messages.
|
| |
|
|\
| |
| |
| | |
'origin/issues/forcouncils/136-show-user-details-for-council-report'
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | | |
'origin/issues/forcouncils/142-allow-anonymous-council-updates'
|
| |/ |
|
|/
|
|
|
|
|
| |
This includes adding/removing reports from a user's shortlist, and
manual reordering of a shortlist with up/down buttons.
The backend code can cope with an item moving to any point in the list.
|