aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * Merge branch 'open311-hook-update-params'Matthew Somerville2019-03-12-38/+59
| |\
| | * [Open311] Use cobrand hook for extra update paramsMatthew Somerville2019-03-08-39/+59
| | | | | | | | | | | | | | | The test has to now create a new comment object each time as `get_cobrand_logged` is cached on the object.
| * | [open311] optionally use fixmystreet_id to match update to problemStruan Donald2019-03-12-2/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If an update has a fixmystreet_id field then use the contents of that in preference to the external_id field to match an update to a problem. This handles the case where a report in a third party system passes through different types (eg inspection and defect), resulting in the external id changing. In this case it is sometimes possible to include the fixmystreet id in each type and hence use that to match things up at the fixmystreet side.
| * | [Oxfordshire] always send easting/northing over open311Struan Donald2019-03-12-4/+2
| | | | | | | | | | | | | | | Remove the config that stopped us sending easting/northing if the user had not clicked the map.
| * | Check better if extra question has values.Matthew Somerville2019-03-12-2/+4
| | | | | | | | | | | | | | | If an Open311 server returned an empty set of values, it was possible for this template to then display a set of empty dropdown options.
| * | [Open311] Proper bodies check for sending updates.Matthew Somerville2019-03-12-5/+4
| | | | | | | | | | | | | | | Otherwise running on a site where one body ID is a substring of another, comments can get processed for the wrong body.
| * | Let all cobrands treat hidden_fields as hidden.Matthew Somerville2019-03-12-2/+2
| | |
| * | [UK Councils] Make sure server_set as appropriate.Matthew Somerville2019-03-12-1/+244
| |/ | | | | | | | | After the change in 1cff2a0e, we need to make sure this is set so that the relevant fields are not shown in the front end.
| * [Bromley] Reduce heatmap default time to one monthMatthew Somerville2019-03-08-1/+1
| |
| * Show all Open311 extra fields in edit admin.Matthew Somerville2019-03-07-3/+1
| | | | | | | | Otherwise any such fields are lost upon a manual edit.
| * Merge branch 'rabx-utf8-encode'Matthew Somerville2019-03-07-4/+61
| |\
| | * Make sure raw RABX column is utf8-encoded.Matthew Somerville2019-03-06-4/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without doing this, a call to e.g. $contact->set_extra_fields(@meta) in PopulateServiceList.pm, with an unchanged meta that contains some Unicode values, can write to the database (and cause an unneeded row in the history table), because the column from the database is UTF-8 decoded, whilst the new text is UTF-8 encoded. It looks like an attempt was made in filter_from_storage to fix this issue, but the column comparison for marking a column as dirty takes place without this being called.
| * | Listen for all inspector form duplicate events.Matthew Somerville2019-03-06-2/+2
| | | | | | | | | | | | | | | | | | The form may be brought in via JS, so we have to add an event to something that will always be there (or alternatively set up when pulled in, which was the old pre-duplicates.js behaviour).
| * | Remove call to removed function.Matthew Somerville2019-03-06-1/+0
| | |
| * | Fix couple of links broken by admin code refactor.Matthew Somerville2019-03-06-2/+2
| |/
| * Merge branch 'stream-csv-output'Matthew Somerville2019-03-05-11/+25
| |\
| | * Stream dashboard CSV output.Matthew Somerville2019-03-05-11/+25
| | | | | | | | | | | | | | | | | | Rather than compile it all into on big string for output, write the output as we go. Also output headers to encourage intermediaries to stream as well.
| * | Merge branch 'issues/commercial/1303-bath-pin-drag-roadworks'Matthew Somerville2019-03-05-70/+48
| |\ \
| | * | [UK Councils] Factor roadworks functions together.Matthew Somerville2019-03-04-69/+47
| | | |
| | * | [BANES] Pin rather than map centre for roadworks.Matthew Somerville2019-03-04-1/+1
| | |/ | | | | | | | | | | | | If the pin is dragged, then category changed, the pin will not be in the centre of the map.
| * | Merge branch '2411-checkbox-gap'Matthew Somerville2019-03-05-25/+11
| |\ \
| | * | Improve inline checkbox spacing.Matthew Somerville2019-03-04-25/+11
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce the space between checkbox and label, so the text isn't lost if the page is zoomed in. Fixes #2411. Replace the creator_fixed question with segmented control to match the full questionnaire, and the admin verified checkboxes with inline-text as they need no padding. Having done that, all uses of .inline are now on their own lines, so no post-padding is needed any longer, but we’ll leave some in case of future use.
| * | Merge branch 'report-links-same-domain'Matthew Somerville2019-03-05-10/+28
| |\ \
| | * | Use relative report links where possible.Matthew Somerville2019-03-05-10/+28
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | On some UK council cobrands, some reports listed might not have been sent to that council, so links to those must go to the national site. However, using absolute URLs for all these reports means that sometimes you change domain when you don't need to (eg. if you’re on osm.fixmystreet.com or an aliased version of the site), which can cause confusion. State when we’re happy to use a relative link (ie. web-facing report links, not emails, or share links) and do that when we can.
| * | Merge branch '374-default-show-name'Matthew Somerville2019-03-05-17/+15
| |\ \
| | * | Default to unchecked for default_show_name.Matthew Somerville2019-03-04-17/+15
| | | |
| * | | Add DateRange class.Matthew Somerville2019-03-04-45/+109
| | |/ | |/| | | | | | | | | | This simplifies the places where a date range is used; it also makes Dashboard date parsing more flexible.
| * | Increase vertical spacing of front page example.Matthew Somerville2019-03-01-2/+4
| |/
| * [Bromley] Better date parsing on heatmap.Matthew Somerville2019-02-28-3/+14
| | | | | | | | IE11 does not support input type=date, so we will get whatever they write.
| * Merge branch '2406-fix-admin-body-dropdown'Matthew Somerville2019-02-28-16/+31
| |\
| | * Fix admin index body dropdown redirect.Matthew Somerville2019-02-26-16/+31
| | |
| * | [Oxfordshire] Limit appears to be lower than 30.Matthew Somerville2019-02-28-4/+4
| | |
| * | [Bromley] Initial heatmap page.Matthew Somerville2019-02-28-11/+623
| | |
| * | [Bromley] Fix colour contrast on active segmented controlZarino Zappia2019-02-28-2/+6
| | |
| * | Do not assign bodyless reports to area IDs.Matthew Somerville2019-02-28-13/+5
| | |
| * | [UK] Add mapzoom to roadworks parameters.Matthew Somerville2019-02-26-0/+1
| |/
| * Merge remote-tracking branch 'origin/issues/commercial/1229-duplicate-reporting'Matthew Somerville2019-02-25-179/+605
| |\
| | * [Borsetshire] Enable duplicate suggestionZarino Zappia2019-02-25-0/+2
| | |
| | * Duplicate suggestion UI when reporting problems.Zarino Zappia2019-02-25-94/+338
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previously staff-only fixmystreet.set_up.manage_duplicates() has been promoted from staff.js to its own file so that we can use it to display the duplicate suggestions on the `/report/new` form. render_duplicate_list (the old refresh_duplicate_list) no longer filters or slices the reports_list returned by the ajax call, since it turns out the server handles all of that (`sub _nearby_json` in `Report.pm` and `sub nearby` in `Nearby.pm`). Since the expandable list items include the "fancybox" image previews, the fancybox CSS now has to be included in the `/report/new` page head, hence the addition of "photo-js.html" in `fill_in_details.html`. The “Get updates” flow reuses the same JavaScript that handles the submission of the "Get updates" drawer at the bottom of around pages.
| | * Internal JSON endpoint to get reports near a pointMatthew Somerville2019-02-25-4/+34
| | | | | | | | | | | | Unused right now, but will be soon, for the duplicate suggestion UI.
| | * Refactor nearby() to use named parameters not listMatthew Somerville2019-02-25-34/+32
| | |
| | * Remove --reports from .item-list--reports__item classes in Inspector UIZarino Zappia2019-02-25-9/+6
| | | | | | | | | | | | | | | There’s no need for this `--reports` to be there, so let’s start phasing it out.
| | * Expandable list item componentZarino Zappia2019-02-25-35/+186
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `_item_expandable.html` is an alternative to regular list items, where you can display extra information on the first click on the item, and then also, optionally, offer a further action, as a separate button. Right now the expandable list items aren’t used anywhere, but we’ll soon be using them in the duplicate report suggestion UI for council staff and the public.
| | * Add space below "map page" contents on narrow screensZarino Zappia2019-02-25-13/+17
| | | | | | | | | | | | | | | Moving the 4em padding from _layout.scss to _base.scss helps separate the content on /report/<id> and /report/new pages from the nav below.
| * | [Buckinghamshire] Tidy up asset attributes.Matthew Somerville2019-02-25-24/+1
| | |
| * | [Buckinghamshire] Add contributed_by to CSV exportMatthew Somerville2019-02-25-0/+82
| | |
| * | Factor out test CSV content function.Matthew Somerville2019-02-25-37/+17
| |/
| * Merge branch 'bugfix/1957'Matthew Somerville2019-02-25-8/+20
| |\
| | * Filters out hidden reports from top 5 list.Joe Siltberg2019-02-25-0/+1
| | |
| | * [FixaMinGata] Adds a cobrand hook (threshold for responsiveness top 5 list)Joe Siltberg2019-02-25-8/+19
| | | | | | | | | | | | Fixes #1957