| Commit message (Collapse) | Author | Age | Lines |
|---|
| ... | |
| | |
| |
| |
| | |
for Zurich tests, count how many reports already exist as a baseline.
|
| | | |
|
| |\ \ |
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This value is now cached, but the template calls the function,
calculating it anew, so for old reports (closed within SLA) the
problem_row.html template showed these as overdue. Instead, we point
the template at a function which first checks the cached value.
(Attempting to roll this functionality into the .overdue() method itself
caused a fair amount of collateral test breakage, and it seems safest to
extract the caching function for now.)
Closes https://github.com/mysociety/FixMyStreet-Commercial/issues/462
|
| | |
| |
| |
| |
| |
| | |
This is required after changes in
59eb44959e9cb8ccd21e9be813e4d1c8baaa1dcc for mySociety internal
deployment toolchain.
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
See https://github.com/mysociety/FixMyStreet-Commercial/issues/461
Again, we just need to substitute the %params line (which elides
hidden reports) for the less restrictive %date_params.
This commit includes very basic tests for the previously untested
?export=1 functionality.
|
| |\ \ \
| |/ /
|/| | |
|
| | | |
| | |
| | |
| | |
| | | |
Test that the slash is added in correct place (the base URL should not
have its own trailing slash).
|
| |/ /
| |
| |
| |
| |
| | |
Remove from admin_base_url (which wraps config vars like
{ADMIN_,}BASE_URL) and add to places in code which aren't
expecting it.
|
| | | |
|
| |\ \ |
|
| | | |
| | |
| | |
| | | |
(for mysoc internal processing of .ugly file)
|
| |\| | |
|
| | | |
| | |
| | |
| | |
| | | |
This is a mySociety internal deployment detail, and should be handled
elsewhere.
|
| |\ \ \
| |/ /
|/| | |
|
| | | |
| | |
| | |
| | | |
So that tests pass on clean dataset such as travis...
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
As per comments after testing mysociety/FixMyStreet-Commercial#459, the
business logic of the actual state changes is fine, but the stats do
also then need to include hidden records.
|
| |\| | |
|
| | | |
| | |
| | |
| | |
| | | |
To allow us to infer additional stats changes requirements such
as in mysociety/FixMyStreet-Commercial#459 better in future.
|
| | | |
| | |
| | |
| | | |
to allow a wider range of action logging.
|
| | | |
| | |
| | |
| | |
| | | |
This was causing previous test failures when config != Zurich, due
to expected logic not getting triggered.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
First test that this page is only available to superuser.
Then test that the moderated and fixed stats are doing something that
is (hopefully) the spec in mysociety/FixMyStreet-Commercial#459
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
see mysociety/FixMyStreet-Commercial#459 the new requirement is for
"Innerhalb von fübeitstagen abgeschlossen" - a report is dealt with
(abgeschlossen) as soon as the public state is Beantwortet (db: fixed -
council or closed) or Unsichtbar (en: Invisible, db: hidden).
Have added this as {closed_overdue} but have retained the logic for
{subdiv_overdue} logged in the %extra hash for now, in case it is
needed.
Note: as per Matthew "as with confirmed/in progress this should be 6 -
it's five days from moderation so is most simply just done as being six
since creation"
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
- is set to 0 on state change within time-limit
- is set to 1 on state change after time-limit
- only marks overdue on first time
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
mysociety/FixMyStreet-Commercial#459 - we move "confirmed" to being
a moderated-but-not-complete state, and run the check for ->overdue
also on changing status from 'unconfirmed' (but only the first time,
so only if $extra->{moderated_overdue} isn't already set.
|
| | | | |
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| | |
see mysociety/FixMyStreet-Commercial#459 - "a report is moderated as
soon as its state has changed for the first time from Erfasst (en:
Submitted / db: unconfirmed) to any other state."
Thus we move 'confirmed' problems from the 1-day moderation SLA, to
a 5-day one, with $include_parent unset (e.g. as with 'planned' it
is with the DM, so doesn't need to be sent to the parent body also.)
|
| | | |
|
| | | |
|
| |\ \ |
|
| | | |
| | |
| | |
| | | |
Omitted the credits and links to further information.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
Use the internal hash URL of the photo so that it can still be seen in
the admin interface.
|
| | | | |
|
| |/ /
| |
| |
| |
| |
| | |
Reduce magic numbers by switching fixed width to a margin elsewhere.
This has the added bonus of making the 'middle-width' .content
have a pleasing right margin.
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | |
| |
| |
| | |
Graphic and css from FMT. Fixes #579.
|
| | | |
|
| | | |
|
| |\ \ |
|
