aboutsummaryrefslogtreecommitdiffstats
path: root/t
Commit message (Collapse)AuthorAgeLines
...
* | [Oxfordshire] Add initials/traffic to RDI desc.Matthew Somerville2017-10-18-16/+17
| |
* | make sure duplicate id provided when marking as duplicateStruan Donald2017-10-17-0/+61
|/ | | | | | | It was possible to mark a report as a duplicate in the inspector panel without providing an id of the duplicate. This prevents that. Fixes mysociety/fixmystreetforcouncils#236
* Merge branch 'sms-mssid-error'Matthew Somerville2017-10-16-25/+25
|\
| * Spot errors coming back when sending text.Matthew Somerville2017-10-12-25/+25
| |
* | log all state changes in admin as commentsStruan Donald2017-10-16-9/+283
| | | | | | | | | | | | | | | | | | | | | | | | Create a comment on a problem when the admin is used to change the state of the problem. If only the state is changed then create a comment with blank text. If the category and state are changing then include details of the category change in the comment. Not all the state changes are displayed at the template level by default. Fixes #1846
* | Merge branch 'bit-of-yaml-tidying'Matthew Somerville2017-10-11-56/+10
|\ \ | |/ |/|
| * Simplify geocode cache read/writing.Matthew Somerville2017-10-10-5/+10
| | | | | | | | Don't think the return data ever needs decoding.
| * Better testing for tests run on live site.Matthew Somerville2017-10-10-51/+0
| | | | | | | | | | | | | | When the tests are run normally, using bin/run-tests, 00-check-config.t is pointless as the config will have been newly set up from the example file. To prevent running except through run-tests, check when test_mode is set, dying if the live config file has been used.
* | Merge branch 'issues/forcouncils/231-multi-ward-reports-view'Matthew Somerville2017-10-09-4/+24
|\ \
| * | Allow ward page to show multiple wards.Matthew Somerville2017-10-06-4/+24
| | | | | | | | | | | | | | | URLs have to be manually constructed, but specifying multiple ward names separated by "|" will show all those wards and their reports on the map.
* | | Fix issue sending alerts around no-text update.Matthew Somerville2017-10-06-1/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | If there was a normal email alert, and then an alert for an update with no text (so no email should be sent), it would still try to send the blank email, and then die because no e.g. $data{cobrand} set. This moves the skip-blank-update check higher up, above any email sending code.
* | | generate update text in alerts if problem state has changedStruan Donald2017-10-05-0/+181
| | |
* | | do not send blank problem update alertsStruan Donald2017-10-05-9/+2
| | |
* | | Merge remote-tracking branch 'origin/fix-geographic-mobile-numbers'Matthew Somerville2017-10-05-4/+4
|\ \ \
| * | | Accept +1 geographic numbers as possibly mobile.Matthew Somerville2017-10-05-4/+4
| |/ / | | | | | | | | | | | | We have no way of knowing whether a +1 geographic number is fixed line or mobile, so accept them all as potentially mobile.
* / / Return page with "No results" if no ref found.Matthew Somerville2017-10-05-8/+15
|/ / | | | | | | | | | | This lookup is often performed by inspectors with appcache switched on, which means they were getting the intercepted Glitch page rather than a 404; give them a no results page instead.
* | correctly handle category changes in inspector formStruan Donald2017-10-04-1/+126
| | | | | | | | | | | | | | | | | | | | | | This resolves two issues when updating the report category in the staff users inspect form: * report category is only updated if the rest of the form is valid * only one update on the report is left Previously changing the category would create an update in addition to any public update created, and the category was always updated even when validation errors occurred elsewhere in the form.
* | correctly handle average state changes of 0 days in area statsStruan Donald2017-10-04-0/+12
| | | | | | | | | | If no problems have changed status then set the average to -1 so we can cope with an average status change of 0 days.
* | move area stats page problems by status calc into databaseStruan Donald2017-10-04-0/+11
|/ | | | | | | | | | | Doing this calculation in code turns out to be much too slow. As part of this also fix an issue where if a report changed state last month but had a further comment this month that was counted as being a state change this month. Also tweak the 'last month' start date to be midnight so the stats don't change throughout the day.
* In RSS, only escape the necessary five/ctrl chars.Matthew Somerville2017-10-02-3/+4
| | | | | | By default encode_entities uses named entities, which don't exist by default in XML. It seemed nicer to switch to only escaping the minimum rather than switching to numeric entities.
* Allow verify/change/add email/phone in profile.Matthew Somerville2017-09-30-8/+143
| | | | | | This expands the 'change email' functionality to work with phone number also, allow addition of missing details, and verifying of unverified details.
* Add ability to confirm reports/updates via text.Matthew Somerville2017-09-30-76/+760
|
* Update admin interface for 'phone' users.Matthew Somerville2017-09-30-49/+129
| | | | | | | This includes search/flagging users, editing user on the report and update edit pages, and user adding/editing itself. Add some more tests for user adding/editing.
* Add ability to log in on /auth via text.Matthew Somerville2017-09-30-27/+146
| | | | A confirmation code is sent via Twilio to be entered on the site.
* Add 'verified' database columns for email/phone.Matthew Somerville2017-09-30-4/+3
| | | | | These are so we can state whether a user's email address or phone number have been verified by confirmation email/text.
* Move some auth functions to their own controllers.Matthew Somerville2017-09-30-121/+128
| | | | | | Move the social auth functions to Auth::Social, and the change email/password functions to Auth::Profile. There are no actual code changes.
* Fix test calling external service.Matthew Somerville2017-09-30-1/+5
| | | | | | This test would pass offline (as query would fail without error), and online was passing because Nominatim was presumably returning no results before today?
* show nearest address for report in inspector panelStruan Donald2017-09-28-0/+37
| | | | | | | If we've stored the nearest address then display that at the top of the inspector panel. Fixes mysociety/fixmystreetforcouncils#234
* set maximum length of report detailed_information in cobrandStruan Donald2017-09-27-28/+48
|
* limit detailed_information to 172 charactersStruan Donald2017-09-27-0/+31
| | | | | | | | RDI files have a 180 character limit for the description so limit the description to 172 to allow for the ID which is also included in this field. Fixes mysociety/fixmystreetforcouncils#228
* area stats page for staff usersStruan Donald2017-09-20-2/+219
| | | | | | | | | Admin page to show some simple summary stats for an area. If the user has been assigned to an area then they will see the stats for that area. Superusers can pick which area they want to view. For mysociety/fixmystreetforcouncils#2
* Fix issues with Open/Closed translated too early.v2.2Matthew Somerville2017-09-15-3/+24
| | | | | | | | | | | | | | It was possible for _hardcoded_states to be reached with no locale selected (e.g. bin/update-all-reports), causing an error trying to translate Open/Closed. But translating those there wasn't correct, because that would then be cached and returned even if a different language was being used. Caching was ignored in testing, which did not help. We no longer translate the state names in their objects, only upon display at the point that we will know the locale. We can't simply return the translation of Open/Closed because there may be entries in the translation table as well.
* Merge branch 'version-2.2'Matthew Somerville2017-09-14-1/+1
|\
| * Version 2.2.Matthew Somerville2017-09-13-1/+1
| | | | | | | | | | Couple of missed strings for translation, and prevent a couple of warnings in tests.
* | [Oxfordshire] Use local time for rdi_processed, not UTCDave Arter2017-09-13-2/+10
|/
* Merge branch 'issues/forcouncils/224-default-priority'Struan Donald2017-09-13-0/+32
|\
| * select default response priority in inspect panelStruan Donald2017-09-13-0/+12
| |
| * edit and display default response priority in adminStruan Donald2017-09-13-0/+20
| |
* | Merge branch '1836-slash-category-bug'Matthew Somerville2017-09-11-0/+1
|\ \
| * | Fix issue editing category with '/' in it.Matthew Somerville2017-09-11-0/+1
| | | | | | | | | | | | | | | | | | Simplify chaining of body/category admin URLs so that all categories are treated the same, with `/` or without, and the template will then always have its CSRF token.
* | | [Oxfordshire] Log when problems sent by RDI email.Dave Arter2017-09-11-1/+32
| |/ |/| | | | | | | | | | | | | The timestamp of when a problem was included in an RDI is shown on the problem’s inspector form. Also check back and include any inspected reports that for some reason haven't been instructed yet. Fixes mysociety/fixmystreetforcouncils#223.
* | [Oxfordshire] tests for response time messagesStruan Donald2017-09-11-0/+52
|/
* sort body dashboard CSV export by report confirmed timeStruan Donald2017-09-07-4/+4
| | | | | And update the tests to avoid a random failure caused by inconsistent ordering.
* Simpler alerts signup page.Zarino Zappia2017-09-06-2/+4
| | | | | | Alert options on the /alert/list page are now in an order that is easier to parse, with a slightly clearer message for users in two-tier locations, and a more prominent email subscription CTA.
* Prevent race condition in root.t test.Matthew Somerville2017-09-06-1/+1
| | | | | | This test assumed /reports would load, which it wouldn't if run in parallel and the data hadn't been generated by reports.t. Test a different page instead.
* include easting/northing in dashboard CSV exportStruan Donald2017-09-04-1/+6
| | | | Fixes mysociety/fixmystreetforcouncils#221
* include ward in dashboard csv exportStruan Donald2017-09-04-0/+26
| | | | | | | Adds a Ward column to the council dashboard CSV export with the name of the ward. Fixes mysociety/fixmystreetforcouncils#220
* Superusers only have permissions available on the current cobrandDave Arter2017-09-01-0/+3
|
* Merge branch 'move-report-states-to-database'Matthew Somerville2017-08-31-91/+155
|\
| * Update form asks for reopen of all closed states.Matthew Somerville2017-08-31-24/+24
| |