| Commit message (Collapse) | Author | Age | Lines |
|
|
|
|
|
|
| |
Restrict email_sign_in token to one day, unused confirmation tokens to
one month. Used tokens will redirect to the created thing but not log
in; don't log in with alert links (unsubscribe link never expires, reply
link will still show "reopen" tickbox).
|
| |
|
| |
|
|
|
|
| |
To allow visible/hidden states to be manipulated at runtime.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Original Report description
- Amended Report description (unsure what this is)
- Media URL (currently one picture, in the future 1-3 pictures)
- Interface used (see open311)
- Requested datetime (see open311) (seems to be same as
created)
- Agency sent datetime (see open311)
- Updated datetime (see open311)
- Council-Response
- title
as per https://github.com/mysociety/FixMyStreet-Commercial/issues/678
|
|
|
|
|
|
|
|
|
| |
Historically, the extra field has been used in two different ways by
different cobrands, both as a list (e.g. Open311 category fields) and a
hash (e.g. the Zurich cobrand).
This commit consolidates usage, adding an API to make use of the field
easier and always returning a hash for the code to use. Fixes #1018.
|
|
|
|
|
| |
This is now handled by normal bodies in the admin. In order to remove
references in send_reports, tidy up (unused) EmptyHomes code a bit.
|
| |
|
| |
|
|
|
|
| |
Fixes #1011.
|
| |
|
|
|
|
|
| |
Rather than redirecting to the report if they're already logged in.
Fixes #1003.
|
| |
|
|
|
|
|
|
|
| |
This includes MapIt postcode lookups, geocoding, query parameters, tile
clicks. Stringifying truncates them to six decimal places, which means
we no longer need any "short" versions anywhere, and the JSON response
will always uses a decimal point regardless of locale.
|
| |
|
| |
|
|
|
|
|
| |
Call the necessary boilerplate in each script so you can call them
directly. Remove boilerplate from files that don't need it.
|
|
|
|
|
| |
This means that e.g. SMTP authentication is used when set up by all
emails, not just non-cron ones. Fixes #988.
|
|
|
|
|
|
|
| |
Includes:
* [UK] Don't show topic form field when reporting abuse.
* Use token in moderation response URL to prevent hidden report leak.
* Make sure successful submission page is full width.
|
|
|
|
| |
Fixes #786.
|
|
|
|
|
|
|
|
|
| |
The body-based alert only works if the body ID matches the MapIt ID.
This fixes #959. Further work needs to be done to enable proper
body-based alerts that work properly in all circumstances.
Consequently, factor out fixed body IDs from many tests.
Also fix a couple of tests not overriding geocoder correctly.
|
|
|
|
| |
Also remove some unused variables.
|
|
|
|
|
| |
This text was being imported encoded, which led to double-encoding when
used in templates. Fixes #960 and #961.
|
| |
|
|
|
|
|
|
| |
Fixes #357.
Also consolidate almost-identical fill_in_details.html template
(for #344).
|
|
|
|
|
|
|
| |
- 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
|
|
|
|
|
| |
Uses LWP::Protocol::PSGI to avoid having to spawn external processes
in order to test these end to end.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
::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)
|
|
|
|
|
|
|
|
| |
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.)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* 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)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
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)
|
|
|
|
| |
Group things together more simply. Fixes #738.
|
|
|
|
| |
This prevents the creation of a category with a blank name. Fixes #556.
|
|
|
|
| |
Looks like Travis doesn't like connections back to itself.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
'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)
|
| |
|
|
|
|
|
|
| |
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.
|
| |
|
| |
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
|/
|
|
|
|
|
| |
Council redirects, BASE_URL comparisons, hard coded links, email
signatures
For #488
|
|
|
|
|
|
|
|
|
| |
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.
|
| |
|