aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet
Commit message (Collapse)AuthorAgeLines
* [IsleOfWight] limit filter category on national around pageStruan Donald2019-09-27-9/+21
| | | | | | | | | Only display non triage categories on this because getting the category expansion to work if we use the triage categories isn't in place and may not be. This is because when we get the list of reports using /around/ajax there's no way to know which body it is for so we don't know how to expand a triage category to the list of 'child' categories. So, just display the non triage ones for now.
* do not send triage comments in alertsStruan Donald2019-09-27-1/+14
| | | | | | add a check into the Alerts script to skip comments that have `triage_report` set in extra. Otherwise we send these our in new_updates alerts.
* [IsleOfWight] correctly filter categories on report pageStruan Donald2019-09-27-8/+105
| | | | | | | | If it's a standard user only display the Triage category list but if one of these is selected then look up all the categories in that group and use those to select the reports to display. It it's a staff user then ignore the Triage categories.
* [IsleOfWight] correctly handle triage categories on reports/around pagesStruan Donald2019-09-27-7/+116
| | | | | | | | | | | | | | Add in some cobrand handlers to make sure a) the appropriate categories are show to staff/non staff users on the around and reports pages b) that those categories are correctly expanded to the child categories when using the category filter to search reports This is because non staff users should only be shown Isle of Wight categories with a send_method of 'Triage', however once those reports are triaged the post triage categories need to be included in any filtering
* [IsleOfWight] add an admin interface to triage reportsStruan Donald2019-09-27-11/+205
| | | | | | | | | | | | | | | | | | On reports with a state of `for_triage` override the `_inspect.html` template to display one which allows only allows the user to change the category from one with a 'Triage' send_method to a non Triage send_method. When saved this records a comment with the original category and the category it was triaged to. This is hidden from the site as it's only used for audit purposes and sending over Open311. This assumes that the triage categories have the same names as the groups of the confirm categories. It uses this to select the first confirm category in the group corresponding to the triage category. This also makes the sidebar geolocation onclick handler check for the presence of the button, as the triage inspect form does not have it.
* [IsleOfWight] add Triage SendReport moduleStruan Donald2019-09-27-0/+20
| | | | sets the state of the report to `for_triage` upon sending
* [IsleOfWight] switch to IoW map tilesStruan Donald2019-09-27-1/+64
|
* [IsleOfWight] skip sending urgent fieldStruan Donald2019-09-27-0/+11
| | | | This is only used to determine if we should block sending the form
* [IsleOfWight] limit admin_user_domain to island roadsStruan Donald2019-09-27-0/+2
|
* [IsleOfWight] display Island Roads for moderation userStruan Donald2019-09-27-1/+5
|
* [IsleOfWight] display external id as ref noStruan Donald2019-09-27-1/+1
| | | | Fixes mysociety/fixmystreet-commercial#1522
* [IsleOfWhite] make sure category_extras uses correct nameStruan Donald2019-09-27-6/+11
|
* [IsleOfWight] configure report sent confirmation emailsStruan Donald2019-09-27-0/+2
|
* [IsleOfWight] include FMS prefix on updates sent over open311Struan Donald2019-09-27-0/+2
|
* [IsleOfWight] fix all reports linkStruan Donald2019-09-27-1/+5
| | | | | | The area name does not match the council name so it redirects to the national site, so check for the all_reports_single_body and use that as the matching instead.
* [IsleOfWight] Open311 fixed update includes message.Matthew Somerville2019-09-27-0/+8
|
* [IsleOfWight] Initial cobrand.Struan Donald2019-09-27-0/+72
| | | | | | | | | Co-authored-by: Matthew Somerville <matthew@mysociety.org> Configured to allow: - updates only from original reporter - close updates from fetch reports - use OSM geocoder
* [Hounslow] Use Confirm validation rules for reportsDave Arter2019-09-23-0/+3
|
* [Open311] Spot <groups> parameter rather than CSV.Matthew Somerville2019-09-20-14/+0
|
* [Northamptonshire] No updates on some reports.Matthew Somerville2019-09-20-0/+19
| | | | Fixed, closed, or reports by the body user.
* Merge branch 'bexley-map-tiles'Matthew Somerville2019-09-19-21/+140
|\
| * [Bexley] Add specific geocoder.Matthew Somerville2019-09-19-4/+74
| |
| * Switch geocoders to be pluggable.Matthew Somerville2019-09-19-16/+26
| |
| * [Bexley] Add new map tiles.Matthew Somerville2019-09-19-1/+40
| |
* | strip quote from contact group namesStruan Donald2019-09-19-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When an open311 category has multiple groups it puts them in the groups element using CSV escaping. This means that group names can sometimes feature double quotes. If we send these to the front end then putting them into the optgroup name attribute breaks the HTML as if the group name is in the database as `"group name"` you end up with <optgroup name="" group="" name=""> instead of <optgroup name="group name"> Hence, stripping the double quotes.
* | [Northamptonshire] Always update report with Open311 state change.Matthew Somerville2019-09-18-0/+11
| |
* | Merge branch 'westminster-com-emails'Matthew Somerville2019-09-17-0/+10
|\ \
| * | [Westminster] No email alerts on .com reports.Matthew Somerville2019-09-10-0/+10
| | |
* | | [Hounslow] Send report to inbox after first Open311 send attemptDave Arter2019-09-16-11/+4
| |/ |/|
* | [Hounslow] Send report to inbox before Open311 send, not afterDave Arter2019-09-13-6/+15
| |
* | Merge branch 'fix-contact-form-auto-bug'Matthew Somerville2019-09-13-1/+2
|\ \
| * | Make sure determine_contact_type called later.Matthew Somerville2019-09-12-1/+2
| |/ | | | | | | | | | | | | The request cobrand is set on the schema during the app's setup_request(), which is called by the root auto action. So anything in a controller's begin action will access an incorrect cobrand (if one has already been set).
* / do not reproject uk wmts map if no coordinatesStruan Donald2019-09-13-0/+2
|/ | | | | | | | If the coordinates are (0,0) then return that straight back as trying to reproject causes Geography::NationalGrid::GB to fall over as 0 lat is outside UK Fixes #2617
* [Bexley] Switch map to OSM.Matthew Somerville2019-09-06-0/+1
|
* Merge branch 'nicer-non-js-category-stuff'Matthew Somerville2019-09-06-15/+53
|\
| * Improve user flow when JavaScript is not availableMatthew Somerville2019-09-05-3/+49
| | | | | | | | | | | | | | This improves the reporting journey to only ask for category, and then category extra questions if appropriate, first, so that if the choice would lead to the form being disabled, this can be shown immediately.
| * Slightly simplify new report category handling.Matthew Somerville2019-09-05-14/+6
| |
* | Merge branch 'issues/commercial/1543-asset-ordering'Matthew Somerville2019-09-05-1/+10
|\ \ | |/ |/|
| * Make sure extras hidden when only_send is active.Matthew Somerville2019-09-04-1/+10
| | | | | | | | | | | | | | | | 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.
* | Allow contact send method to be unset always.Matthew Somerville2019-09-05-1/+1
|/ | | | | | 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.
* Allow more characters in name splitting.Matthew Somerville2019-09-03-1/+1
| | | | Not sure how this has not arisen before!
* [UK Councils] Config specifying of update closure.Matthew Somerville2019-08-30-11/+58
| | | | | 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.
* Merge branch 'issues/1515-disable-form-flag'Matthew Somerville2019-08-29-0/+30
|\
| * Allow form-disabling per Open311 question answer.Matthew Somerville2019-08-29-7/+18
| |
| * Add admin to disable reporting form per category.Dave Arter2019-08-29-0/+19
| | | | | | | | | | Remove all existing hard-coded register_category calls, and have a new way of implementing staff_ignore.
* | Make sure anonymous reports are marked anon irrespective of permissionsDave Arter2019-08-29-0/+2
|/ | | | | | | | | | | | | | 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.
* Merge branch 'issues/commercial/1514-extra-data-protected'Matthew Somerville2019-08-29-0/+2
|\
| * [Open311] Add 'protected' category extra flag.Struan Donald2019-08-23-0/+2
| | | | | | | | | | If an extra field on a category has a protected flag then do not overwrite or remove it when populating categories.
* | [Hounslow] Add pin_colour functionDave Arter2019-08-27-0/+6
| |
* | [Buckinghamshire] Fix road placement question bug.Matthew Somerville2019-08-27-1/+1
|/ | | | | 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.