| Commit message (Collapse) | Author | Age | Lines |
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
|
|
| |
Fixes #357.
Also consolidate almost-identical fill_in_details.html template
(for #344).
|
|
|
|
| |
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.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
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.
|
| |
|
|\ |
|
| |
| |
| |
| | |
And FixMyStreet.com specific open questionnaire page.
|
|/
|
|
|
|
|
|
|
|
|
| |
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.
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Add some comments to the top of the open311 bin scripts.
|
|
|
|
|
| |
Otherwise oddities arise due to summer time differences
putting a BST now-6d23h time behind a UTC now+1s-1w.
|
|
|
|
|
| |
This is very old, no-one seems to want/need it any more,
and more than one reuser does not want it. Fixes #715.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
| |
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.)
|
|
|
|
|
| |
The call to set_lang_and_domain needs to happen before
the first string is translated (the report state).
|
| |
|
|
|
|
|
|
| |
Put alerts in default cobrand, so changes to other cobrand email
templates don't affect the test; and clean up at the end of the
test so that it can be rerun without issue.
|
|
|
|
|
| |
Test that the slash is added in correct place (the base URL should not
have its own trailing slash).
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a body has been marked as deleted (currently only exposed in the
Zurich cobrand's admin) the categories for that body should not be
available on the report_new form.
Prior to this change there was an error running the
'zurich-overdue-alert' script, because it wasn't expecting the
'bodies_str' field to have any entries in the format 'x|y,z', this
prevents this problem from happening in the future (though it doesn't
fix any existing entries with this issue).
|
|
|
|
|
|
|
|
|
| |
If the ALLOWED_COBRANDS configuration variable only contains one entry
(and also work if it's a string rather than a list), always use that
cobrand, no matter what the hostname is.
The example Vagrantfile no longer needs the configuration changes at
all.
|
|
|
|
| |
Use a relative URL in the CSS in case it's proxied.
|
|\ |
|
| | |
|
|\ \
| |/
|/|
| | |
https://github.com/andylolz/fixmystreet into andylolz-389-report-search-ref
|
| | |
|
| | |
|
| |
| |
| |
| | |
This fixes #527.
|
|/ |
|
|
|
|
| |
Fixes #586
|
|
|
|
|
| |
This is for consistency as Sub::Override is used for another test and
also so we minimise the number of modules installed just for testing
|
|
|
|
|
|
|
|
| |
The first email count test in the abuse subtest was failing on Travis
but not on my machine because the sent email alert test was running on
my machine, as it has the mock module installed, and this was resetting
the bodies_str to one with a confirmed contact. Setting bodies_str on
both these tests stops this sort of thing happening again
|