| Commit message (Collapse) | Author | Age | Lines |
... | |
| | | |
|
|\ \ \
| |/ /
|/| | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The extras may be a confusing stopper message that only applies to the
'main' body, or it may be extra questions that are now not required.
Either way, we should hide this information, and we need to make sure
the server no longer checks if required extra questions have been
answered.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This was previously restricted to coming directly to /report/new, but
this meant it wasn't being run when e.g. Buckinghamshire winter routes
or Westminster TfL roads were being loaded after the appropriate
category selection, so their effects did not occur (as the checkFeature
code was called prior to the layer finishing loading).
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The stopper check runs first, which could mean that if the asset layer
has not yet loaded, the form is disabled when it should not be (e.g. on
a TfL road in a road category). Make sure that we always run the stopper
check after the TfL layer check, in case it changes something.
|
| |/ |
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
If a body was set to Open311, and a contact set to Email, it was
impossible to unset the contact's send method and set an Open311
code, because it would always demaned a valid email address.
|
|/ /
| |
| |
| |
| |
| | |
It's still not working as expected, with lots of rejected reports.
This reverts commit 49c74ded4275005e766a948ec47d4cc722e0567e.
|
|/ |
|
| |
|
|
|
|
|
|
|
|
| |
Note how in road_not_found, the Highways England check could never pass
because the only_send was being removed the line before. When the
Buckinghamshire and Northamptonshire code was factored together, it
looks like bits were taken from one that weren't present in the other,
and so should not have been present in the shared code.
|
|
|
|
|
| |
Any message being included there will be for the appropriate council and
not relevant.
|
|
|
|
| |
Not sure how this has not arisen before!
|
| |
|
| |
|
| |
|
|
|
|
| |
Fixes mysociety/fixmystreet-commercial#1293
|
| |
|
|
|
|
|
| |
A cobrand feature flag can be used to only allow updates on open
reports, by the original reporter, to staff only, or to no-one.
|
|\ |
|
| | |
|
| |
| |
| |
| |
| | |
Remove all existing hard-coded register_category calls, and have
a new way of implementing staff_ignore.
|
| | |
|
| |
| |
| |
| | |
For mysociety/fixmystreet-commercial#1496
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
| |
There was a small bug where the 'default_to_body' permission would
override the 'report anonymously' button when staff users were adding
reports to the site. The result of this was that the name of the
anonymous user record would be shown. No harm done, as that user's name
would likely be set to "Anonymous user" in config, but it resulted in
reports pages showing the slightly odd wording along the lines of:
Reported in the Bin bags category by Anonymous user at 14:57 today
This commit ensures the contributing_as_body flag is mutually exclusive
with the 'report_anonymously' request parameter.
|
|\ |
|
| |
| |
| |
| |
| | |
If an extra field on a category has a protected flag then do not
overwrite or remove it when populating categories.
|
| |
| |
| |
| |
| |
| | |
The asset data we hold is out of date with Confirm, and is causing
problems sending reports as the central_asset_id field is being
rejected by Confirm.
|
| | |
|
| |
| |
| |
| |
| |
| | |
The CSV utils function returns an array of arrays, not just an array,
which this code did not realise. All this affected was sometimes both
bodies were shown in a two-tier area when the code meant to show one.
|
|/
|
|
|
| |
The question was being asked for even when it should not, because
the error key had been changed to start with an 'x' in 120b57d1.
|
| |
|
| |
|
|\ |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If a message is sent via the contact form, with a stated email address
matching a user in the database, the resulting email will now include
quick links to:
- Admin page for editing the user
- Admin page for viewing the user’s reports
And if the user has made at least one report, also:
- Admin page for the user’s most recent report
This should speed up admin email triage considerably.
|
| | |
|
|/ |
|
|
|
|
| |
The app checks if name is set as a test for successful login.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
These were being double-escaped, but c.uri_for returns a string that’s already URI-safe
|
| |
|
| |
|
| |
|