| Commit message (Collapse) | Author | Age | Lines |
|
|
|
| |
[Bristol] Use more zoomed-in default link zoom.
|
|
|
|
| |
Fixes a bug with updates not being displayed correctly in the app.
|
|
|
|
|
| |
Also takes the opportunity to fix the incorrectly-encoded sender name which
was appearing to some clients as "Züri Wie Neu".
|
|
|
|
|
|
|
|
|
| |
- Adds is_superuser flag to User
- Logged-in user must be a superuser or have from_body set in order to access
anything within /admin
- has_permission_to on a superuser will always return true
- Only superusers can create/grant superusers
- New `createsuperuser` command for creating superusers
|
| |
|
| |
|
|
|
|
|
|
|
| |
Historically, emails sent offline (alerts, questionnaires, etc) used a
different templating system from those sent by the website (e.g. login
emails), though the newer system was also being used for the site name
and signature of offline emails.
|
|
|
|
|
|
|
| |
Store image type along with hash in photo column, and use that when
outputting images / generating URLs. Make sure all public photo URL
generation goes through appropriate functions, and change temp URLs
so the filename can be output directly from the list.
|
| |
|
|
|
|
| |
Fixes #1337.
|
|
|
|
|
| |
Remove get_photo_params, which only looked at the first photo,
make explicit when we're doing that using `.first`.
|
|\ |
|
| |
| |
| |
| | |
Command line scripts don't need a full blown app, just database.
|
| |
| |
| |
| |
| | |
Some categories ask the user for extra information such as street or house
number; these are output in the CSV if the report includes them.
|
| | |
|
|/ |
|
| |
|
|
|
|
|
|
|
|
| |
The 'public response' to a problem was being omitted from the email sent to the
user when the problem is closed, because the email was sent before the report's
'extra' field was updated with the response body text.
This commit sets the public response on the problem before sending the email.
|
|
|
|
| |
See mysociety/FixMyStreet-Commercial#674, mysociety/FixMyStreet-Commercial#673
|
|
|
|
|
|
| |
On the report admin page, include the message the user received when the report was closed.
See mysociety/FixMyStreet-Commercial#718
|
|
|
|
| |
See mysociety/FixMyStreet-Commercial#690
|
|
|
|
|
|
|
| |
- Only send external_message when the problem state has the correct value
- But make sure the message is preserved for editing when problem state is 'planned'
See mysociety/FixMyStreet-Commercial#718
|
|
|
|
|
|
|
|
|
|
|
| |
- Styling for "Updated!" message
The message shown on form submission no longer pushes both columns
down, and disappears after 5 seconds.
- Make "time spent" labels inline to save space
- Simplify display of report admin category dropdown
- Change order of report admin inputs for Division Managers
See mysociety/FixMyStreet-Commercial#690
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
See mysociety/FixMyStreet-Commercial#690
I understand redirection to summary page was introduced at ZWN's request,
to make it easy to
process reports and then quickly move onto the next one.
However, during testing now, Tobias has mentioned this is a) slowing
things down and b) confusing because it's not obvious that the report
has been saved.
I've tried to address (b) by adding the "Aktualisiert!" message when
you are redirected.
Also, for (a) I've removed the redirection from a few cases, and
disabled it for superuser. If need be, I can remove redirection from
more (or all) cases.
Fix Official answer/Reply to user
- hide label for status update on state change
- correct wording on button for closure (single wording)
- correct wording on label for user reply (either "Official answer" or "Reply to user" as appropriate)
- Make sure the official response texts are shown for edit/static as appropriate, and test.
- javascript improvements
- honour public status update for Extern/Wunsch too
- don't show public message for Wunsch
- Ignore all other fields when rotating photos.
(See mysociety/FixMyStreet-Commercial#718)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Requirements:
=============
1: DM to be able to mark a problem as Wish, Extern, Not contactable,
Hidden, Jurisdiction unknown
2: SDM to be able to mark a problem as Not contactable.
3: DM to be able to add information to such a problem, and trigger
emails to external agencies, and to user, closing the problem at the
same time.
4: Closed reports must be categorized as Wish/Extern/NC/Hidden/JU for
statistical reporting
5: Non-closed reports must appear as open, to allow DM to see them in
the list, similar to what is currently done for Ruckmeldung Ausstehend
(Feedback outstanding).
5b: however "Hidden" problems must be immediately hidden from public
view.
Proposal:
=========
a) Requirement 1 will be carried out in exactly the same way by
DM. e.g. Category dropdown select
b) Requirement 2 will be carried out in exactly the same way by SDM.
e.g. "Not contactable" button
c) BUT, for a), c), the category will be diverted to "Ruckmeldung
Ausstehend" (Feedback outstanding), and metadata will be saved with the
appropriate closure type.
d) NOTE: this means the report will still be visible to public as "In
bearbeitung" (Except for Hidden reports, as per 5b)
e) The extra fields will be presented on the "Ruckmeldung Ausstehend"
page, just as defined in the PDF, depending on the metadata. e.g.
- Wish: Zustaendige Stelle + Message
- Extern: External body + Message2
- Not contactable: (no extra info)
- Hidden: (no extra info)
- Jurisdiction unknown: (no extra info)
f) When the DM publishes the answer (e.g. "Antwort Veroeffentlichen"
button), the state will normally change to Fixed. BUT in the new cases,
the metadata saved in step c) will be checked, and the report will
change state to Wish/Extern/Not contactable/Hidden/Jurisdiction unknown,
as appropriate. The report will now be visible/hidden as previously
discussed depending on the new state.
g) If the report is edited again (in /admin/report_edit) it will be in
the relevant state, and any extra fields will be visible as per e).
Those fields may be updated.
Alternatively, the state may be changed, possibly triggering a) again,
and clearing any metadata.
- Fix javascript interactions 690
- show new statuses in summary
- fix photos on SDM page
- fix Planned => Planned+metadata transition
- stop "assignation .select" js affecting templates
- fix redirection for Extern and generally
See mysociety/FixMyStreet-Commercial#672, mysociety/FixMyStreet-Commercial#690
|
|
|
|
|
|
|
| |
- submit-external-wish.txt includes a link with report's coordinates, template values are interpolated
Conflicts:
perllib/FixMyStreet/SendReport/Zurich.pm
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
See mysociety/FixMyStreet-Commercial#690
- external_message for Extern/Wunsch states
- as per PDF mockups, the "public response" field is now editable
from more states, namely:
- Ruckmeldung ausstehend
- Extern
- Zustandigkeit unbekannt
- Wunsch
- Nicht kontaktierbar
- Unsichtbar
- test fixes
NB: slight hack to reorder Zurich report_display tests
As these remove 'investigating' from hidden_states, these break
tests for default cobrand (the FMS::App code expects to be called
once on request start, not multiple times within one test)
- Not contactable button
and tests for this and the similar `send_back` functionality
(previously untested).
NB: Unsure why we need to specify form_number=2 just for these submits,
but test won't find the buttons otherwise.
- Make sure SDMs' "time_spent" figure is recorded.
(See mysociety/FixMyStreet-Commercial#718)
|
|
|
|
|
|
|
| |
Have moved the generation of banner text from the template into
problem_as_hashref
See mysociety/FixMyStreet-Commercial#672
|
|
|
|
|
|
|
|
| |
See issue in mysociety/FixMyStreet-Commercial#663 with
problems not appearing in the "Erfasst" filter. It looks like Zurich
requires a single id in the `bodies_str` (like '4'), and may rely on the
configuration of bodies/contacts in database to be perfect to avoid them
becoming something like '4|47,48'.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mechanism is general, but current implementation only used
for Zurich.
- Zurich: Record time_spent field mysociety/FixMyStreet-Commercial#676
- template changes (including jQuery spinner)
- backend changes to save data are made:
- in log_edit, if supplied
- and time on category change (and arbitrary actions)
- Add update_admin_log to all paths for DM, ensuring that it gets called,
while resetting the `time_spent` parameter, to make sure that
there is never a duplicate record created.
- Also quell warning about using DateTime values in query.
- Also add (seemingly unrelated) parens to stop misparsing of OK line in
questionnaire test now that DT::Format::Pg is used in Zurich cobrand
See mysociety/FixMyStreet-Commercial#676
|
|
|
|
|
|
|
| |
This provides a ResponseTemplate model that an admin/staff
user can choose from when responding to problems.
For mysociety/FixMyStreet-Commercial#671.
|
|
|
|
| |
See mysociety/FixMyStreet-Commercial#675
|
| |
|
|
|
|
|
| |
For mobile, but should be done via same mechanism.
See mysociety/FixMyStreet-Commercial#670
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
for Zurich (and generic data), progressing
mysociety/FixMyStreet-Commercial/issues/663, to provide a way
for body/contact config to be consistent across dev/staging/live.
Run like so:
bin/setup-contacts zurich
e.g. with the lower-case sigil describing the cobrand to run
setup-contacts for.
|
|
|
|
| |
See mysociety/FixMyStreet-Commercial#669
|
|
|
|
|
|
|
| |
Admin CR4 https://github.com/mysociety/FixMyStreet-Commercial/issues/673
seems like admin_pages should be able to generate the menu, but
this change also needs template change.
|
|
|
|
|
| |
This override is not used anywhere, it dates from a very old long-gone
test script, and causes issues with database query planning.
|
|
|
|
|
|
|
| |
Use a central get_param and get_param_list functions dependent on
whether we're after a scalar or a list (almost always a scalar). This
prevents any possibility of confusion where param() could return a list,
or params->{} an arrayref.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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.
|
|
|
|
|
|
| |
Instead, add it if not present at the negotiation stage. Even if not
negotiated, we currently need it to be present for the in_gb_locale()
calls to work.
|
| |
|
|
|
|
|
| |
This is very old, no-one seems to want/need it any more,
and more than one reuser does not want it. Fixes #715.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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.
|
| |
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
| |
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
|