aboutsummaryrefslogtreecommitdiffstats
path: root/t
Commit message (Collapse)AuthorAgeLines
* Don't generate unused special templates/po files.Matthew Somerville2014-11-19-2/+2
|
* Show logged in message as success, not error.Matthew Somerville2014-11-18-8/+7
| | | | | | Fixes #357. Also consolidate almost-identical fill_in_details.html template (for #344).
* [Warwickshire] Integration bits during/after visitHakim Cassimally2014-10-16-184/+220
| | | | | | | - Tweaks required to get things working in WCC environment - update Open311 to onsite changes - updated Open311 parameters after WCC's Bentley and County Highways workshop. - ... including ce_cpr_id
* Open311: FMS <-> Open311 Endpoint End to end testsHakim Cassimally2014-10-16-9/+147
| | | | | Uses LWP::Protocol::PSGI to avoid having to spawn external processes in order to test these end to end.
* Open311 Warwick (Exor) IntegrationHakim Cassimally2014-10-16-8/+330
| | | | | | | | | | | | | | | | | | | | | | | ::Integration::Warwick subclasses ::Integration::Exor refactor request_class and new_request Exor service Warwickshire updates retrieval, with datetimes stubbing out of Oracle constants, for local testing We also edit FMS's core PopulateServiceList routine to hide system fields from FMS: Bromley/Warwickshire send metadata in their services/FOO.xml advising that you can pass, e.g. attributes[easting]. FMS by default shows all of these to the user to fill in, however we don't *want* the user to supply these, rather they are added by the cobrand. Bromley had an exception for this (keyed by $body->areas->id). We write this more generally for Warwickshire too, keying instead by $body->name (as this is far less likely to be overridden for installs using global or custom Mapit's)
* Open311 Role for accepting default config fileHakim Cassimally2014-10-16-0/+24
| | | | | | | | See also MooX::ConfigFromFile, but that's underdocumented and seems overengineered -- may be worth implementing if requirements become more complex however. (See also Config::Any, which is well worth doing in future, using YAML only reflects current usage in FMS though.)
* Open311 Endpoint mySociety extensions roleHakim Cassimally2014-10-16-3/+194
| | | | | | | | | | | | | | * Get Service Request Updates This requires a new object ::Service::Request::Update, which of course is not part of standard spec. So, in order to make the core not too contaminated by : * the endpoint should instantiate ::Service::Request::mySociety objects which know about updates * have added a learn_additional_types callback from Schema to Endpoint, so that core doesn't need to know about /open311/service_request_update * (but ::Spark knows about the exception for updates... meh, but is 1-line)
* Open311 EndpointHakim Cassimally2014-10-16-0/+623
| | | | | | | | | | | | | | | | | | | | | Subsystems include * ::Spark encoding conventions for xml/json * ::Schema using Rx to validate form of inputs and outputs, including validation for, e.g., dates and CSV as part of Open311 Handles following paths: * Open311 attributes for Service Definition http://wiki.open311.org/GeoReport_v2#GET_Service_Definition * POST service request * GET Service Requests * GET Service Request Objects: * ::Service * ::Service::Request
* [Warwickshire] cobrand workDave Arter2014-10-16-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First Warwickshire styling - imported WCC's header and footer files - Use Warwickshire colours for page elements - Include FMS elements in header - Use correct Warwickshire colours on map - Use full-screen map on desktop. - Improve display of homepage elements - Ensure correct classes are added to body tag - Make sure site header has correct height on front page - Tidy-up of unused header content and wrappers - Improve display of map - Position map beneath other elements Improve map display - Take some CSS from Warwickshire styles to ensure its applied - Fix container widths on map page - Make sure map wrapper is the correct width - Add style overrides to match Warwickshire.gov.uk - Ensure map controls are visible Nav and mobile styling - Make sure sticky sidebar nav is usable - Improve mobile front page - Ensure map is full-screen on mobile - Improve front page display on mobile - Fix map display on desktop - Fix Warwickshire and FMS nav on mobile - Add viewport meta tag for mobile devices - Tighten up vertical spacing on front page - Fix header height on tablet size screens - Set pan/zoom position in CSS, not JS Styling tweaks - correct styling of nav for wider screens (previous hidden by media query) - Add draft Warwickshire FAQ - Fixes https://github.com/mysociety/FixMyStreet-Commercial/issues/611 - Warwickshire. remove all reports link as requested. - see https://github.com/mysociety/FixMyStreet-Commercial/issues/609 - Update WCC footer - see https://github.com/mysociety/FixMyStreet-Commercial/issues/597 Bring WCC css locally, and override - Override for https://github.com/mysociety/FixMyStreet-Commercial/issues/612 - favicon https://github.com/mysociety/FixMyStreet-Commercial/issues/616 - update contact details - custom FAQ - fix padding on mobile https://github.com/mysociety/FixMyStreet-Commercial/issues/617
* UK. Generic problems_clause for two-tier councilsHakim Cassimally2014-10-10-0/+28
| | | | | | Replace copy-pasted problems_cause declarations in Hart, Oxfordshire, and East Sussex with a generic override in UKCouncils.pm (as a bonus also overrides Stevenage)
* Prevent overhang of admin body categories table.Matthew Somerville2014-10-08-7/+7
| | | | Group things together more simply. Fixes #738.
* Validate category name/email/note in admin.Matthew Somerville2014-10-08-0/+1
| | | | This prevents the creation of a category with a blank name. Fixes #556.
* Couple of tests missing MapIt override.Matthew Somerville2014-10-08-2/+10
| | | | Looks like Travis doesn't like connections back to itself.
* Update test for changes in 6eff7846.Matthew Somerville2014-09-11-2/+2
|
* Tests for moderationHakim Cassimally2014-08-14-0/+391
|
* [SeeSomething] fix and test for stats contentHakim Cassimally2014-07-31-1/+50
|
* Replace from_council with from_body in SeeSomething cobrandSteven Day2014-07-31-0/+39
| | | | | | | | | | | | | | | | | | | | | | 'from_council' was renamed to 'from_body' and turned into a relationship in b2aa34b Feb 2013, and the seesomething code was missed. Though the FMS::DB::Result::User object would (correctly) throw an error if an invalid method is called, *Catalyst* idea of a user object wraps this in a Catalyst::Authentication::Store::DBIx::Class which uses AUTOLOAD to delegate methods and swallows the error. e.g. this code always returns false, hence the code path is never run. (I think this is a bug and have opened: https://rt.cpan.org/Ticket/Display.html?id=97640 ) Added test case and (hopefully) corrected template, will need a check against site running copy with recent data to check that nothing looks squiffy. We also had to change the %council_details stash, which was based on the old code, not sure if suggested fix matches original intent, but looks plausible. (As discussed with @dracos @stevenday this is slightly repeated, and should arguably use ->{bodies} but that's a possible future FIXME)
* Removal old Southampton/Reading/Lichfield cobrands.Matthew Somerville2014-07-29-52/+52
|
* Some tweaks for Travis timeout errors.Matthew Somerville2014-07-29-7/+12
| | | | | | Hopefully this will reduce the chance of a timeout: * Wrap admin body tests in MapIt override. * Use full hostname to save a redirect each time.
* Switch test to use different working cobrand.Matthew Somerville2014-07-14-8/+9
|
* Update GA code to use ecommerce ping.Matthew Somerville2014-07-08-3/+7
|
* Merge branch 'mysociety/798-show-reports-subset'Matthew Somerville2014-07-04-0/+118
|\
| * Link through from all reports page to sub categoriesStruan Donald2014-07-01-0/+118
| | | | | | | | | | | | | | | | Add a type parameter to the /reports/body page to restrict list by problem category. Only link to report categories that have entries. Fixes #798
* | Use https for UK site where appropriateStruan Donald2014-07-03-2/+2
|/ | | | | | | Council redirects, BASE_URL comparisons, hard coded links, email signatures For #488
* Better spotting of signing in on /auth form.Matthew Somerville2014-06-20-1/+19
| | | | | | | | | If your browser autocompleted form fields, you could fill in the signing in part of the form but still be sent a confirmation email. This commit will now default to trying to sign in if the sign in button is clicked or there is data in the signing in password field. Fixes #816.
* Fix test of postcode that has moved ever so slightly.Matthew Somerville2014-05-30-9/+9
|
* Merge branch '700-auto-emails'Matthew Somerville2014-05-16-19/+134
|\
| * Update all email templates with new text.Myfanwy Nixon2014-05-16-19/+134
| | | | | | | | And FixMyStreet.com specific open questionnaire page.
* | Add MAPIT_GENERATION variable, to pin whitelist.Matthew Somerville2014-05-16-1/+3
| | | | | | | | | | | | Otherwise, when MapIt Global is updated, the 'point' lookup, as it only returns the current generation by default, might no longer return the areas present in the whitelist.
* | Test dashboard CSV export more thoroughlyDave Arter2014-04-30-3/+13
|/ | | | | | | | | | | Instead of counting the number of lines output by dashboard CSV export, the content is parsed as CSV and the number of rows present is checked. Also includes a problem report that's split across multiple lines, testing the fix for mysociety/FixMyStreet-Commercial#494 introduced in d75e161c151c4f1e6dced24eae3170e01f9b7ba6. Closes #780.
* Merge branch 'improve-contact-form'Struan Donald2014-04-17-1/+110
|\
| * Add a who are you trying to contact option to contact formStruan Donald2014-04-15-1/+110
| | | | | | | | | | | | | | | | | | | | | | In order to try and cut down on people mistakingly using this form to try and contact their council all some 'who are you trying to contact' options to the form that display appropriate messaged unless the 'FMS team option is selected. Add the validation code for this in the cobrand module. Fixes #41
* | alter pave url sent to analytics on report completionStruan Donald2014-04-16-1/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to allow us to do funnel analysis in google analytics we need to send a differnt page url when a user completes a report. To do this we add a report_created=1 to the query string on either the report page or the report confirmation page for logged in and confirm by email users respectively. There's a bit of complication as we don't want to set a session cookie everywhere as that will break caching so we set a stash variable, although if the user is logged in this is controlled by a variable in the flash as there is already a session cookie at this point. also, removes the code that sends an event upon report completion as this was no use for funnel analysis.
* | create westminster council so report_new test can run standaloneStruan Donald2014-04-16-0/+1
| | | | | | | | | | | | | | The test relies on a body for westminster council existing which is crated in an earlier test if you run the whole test suite. However it's not present if you run only this test against an empty database so add it to the list of created bodies.
* | Add Cobrand config to include update date in email alertsStruan Donald2014-04-08-0/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | This is useful for sites where the updates aren't published on the site so you need a bit more context in the email alerts. Add inlude_time_in_update_alerts to default cobrand, off as default If this is set then the date is added after the name string in the email, passing in 'alert' as the type to dt_prettify in order to allow the cobrand to format it accordingly. This does require pulling out the alert confirmed time from the database in the send_alerts code where it wasn't before.
* | Allow comments to be confirmed with no confirmationStruan Donald2014-04-08-0/+79
| | | | | | | | | | | | | | | | Add never_confirm_updates setting to cobrand that, when set to 0 means that comments are confirmed with no confirmation step. Essentially behaves as it would if the user was logged in. Also add a flag to the flash that indicates the comment was accepted to you can display a message on the report page if required
* | Add some overrides to zurich tests to stop failureStruan Donald2014-04-08-0/+4
|/ | | | Make sure mapit_id_whitelist set for tests that submit
* Mock a date for dashboard testsHakim Cassimally2014-04-07-1/+5
| | | | | | | Fixes #365 (how appropriate) by mocking datetime to allow the dashboard test's assumptions (that we're not so near beginning of year that a report from X days ago isn't still in same year) to hold.
* Update old-style open311 script to use bodies.Matthew Somerville2014-04-07-36/+33
| | | | Add some comments to the top of the open311 bin scripts.
* Use local DateTimes in dashboard and its tests.Matthew Somerville2014-04-07-6/+7
| | | | | Otherwise oddities arise due to summer time differences putting a BST now-6d23h time behind a UTC now+1s-1w.
* Change default email indentation to none.Matthew Somerville2014-03-17-17/+17
| | | | | This is very old, no-one seems to want/need it any more, and more than one reuser does not want it. Fixes #715.
* Fix slight regression in cf20c709, to still match all UK cobrands.Matthew Somerville2014-03-13-0/+1
|
* Add test for de47a7f.Matthew Somerville2014-03-12-15/+63
| | | | | | This tests that the default string used by the old-style Open311 update code if there is no incoming string and the report is marked fixed, is translated.
* Merge remote branch 'origin/sweden-rebase'Matthew Somerville2014-03-11-0/+200
|\ | | | | | | | | | | | | Conflicts: bin/update-schema locale/sv_SE.UTF-8/LC_MESSAGES/FixMyStreet.po perllib/FixMyStreet/DB/Result/Body.pm
| * Alter Open311 description to only include English on FixMyStreet.Jon Kristensen2014-03-11-0/+2
| | | | | | | | This could be made cobrand-customisable in future.
| * Bring back open311-update-reports, GetUpdates.pm and getupdates.tJon Kristensen2014-03-11-0/+198
| | | | | | | | | | | | This commit reverses the "Remove unused Open311 GetUpdates code." commit. FixaMinGata, the Swedish FixMyStreet cobrand, is still using open311-update-reports.
* | Export reports as CSV on /dashboardHakim Cassimally2014-03-10-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provide a button to download the details and text of reports at the second "Look up" button on /dashboard. Provides acknowledged, fixed, closed dates by parsing updates. Fixes #692 (and mysociety/FixMyStreet-Commercial#399) Email is not shown, and name honours anonymous flag: as /dashboard is accessible to people who are from_body (who still do not see anon-users name and email in the FMS web app.) A basic test is provided as per Zurich's similar report.
* | Re-enable dashboard testsHakim Cassimally2014-03-10-519/+521
| | | | | | | | | | | | | | | | The tests have been commented out due to long-running failure. It looks like the failures happen if working copy MAPIT_URL is not pointing at live site. Have updated with an override_config setting this to live site (which is obviously suboptimal, but seems to be widespread practice across t/ at the moment.)
* | [Hart] basic Hart cobrandingHakim Cassimally2014-03-05-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally copied from template of Bromley code, substituting logo, council ID, geolocation data, and renaming .scss variables etc. (NB: this was a bad idea. As per notes/cobranding.txt, should have started with less copying of files, as this required more cleanup. Bromley's code has some particularities, like smaller map, split First/Last name fields and so on. The current branch has been rebased to present a slightly idealised picture of the process.) (Includes contributions by @davea and @stevenday) * Some code cleanup and added is_two_tier * Add Hart-specific links to footer * Shift FMS nav out of the main header * Update styling to follow design more closely * Flip nav colours to better indicate links * Make header banner entire width of the page * Stretch footer bg to entire width of page * Add top-level nav links * Correct header height on mobile * Correct wording of top menu item * Remove FMS logo from top of page * Shift top level nav to correct position * Invert media query so IE8 displays header properly * hide folding triangle, as makes no sense on this cobrand * Some more renaming of Bromley to Hart * Treat Hart the same as Bromley in the JS * Whitespace/formatting fixes * Basic test for cobrand * Javascript #map_box fixes * Make sure #map_box is not in desk-only * ensure there is only single map (one .wrapper only) * colour tweaks * Fix example places (postcode, spelling) * Basic FAQ text * Set Zoom level 3 * CSS/HTML/JS tweaks removing Bromley customizations * Switch templates to put navigation in footer, as FMS.com does * remaining HTML/CSS changes for large map * replace favicon * move site-logo scss to one place * make small logo transparent
* | Merge branch 'issues/677-open311-errors-on-comment-processing'Matthew Somerville2014-02-27-9/+25
|\ \
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-26 08:40:08 +0000