Commit Graph

8333 Commits

Author SHA1 Message Date
karlchillmaid
ebf5ce5a76 Update printing and dispatch information
Update printing and dispatch information to match status description
2019-09-18 15:07:18 +01:00
karlchillmaid
27421a2556 Update status description 2019-09-18 15:04:05 +01:00
karlchillmaid
d6dffff03c Update status description 2019-09-18 14:55:25 +01:00
karlchillmaid
75bc716fd5 Update content 2019-09-18 14:52:58 +01:00
karlchillmaid
ba3935f45e Add 'Printed' status 2019-09-18 14:50:38 +01:00
Katie Smith
e18e2d1e98 Merge pull request #3121 from alphagov/upload-precompiled-letters
Upload precompiled letters
2019-09-12 16:09:07 +01:00
Katie Smith
81da8e762d Update for new template preview sanitise response 2019-09-12 09:54:36 +01:00
Katie Smith
79053dec93 Allow uploaded letters to be sent if valid
Added a send button which only appears on the page if the query string
indicates that the PDF is valid. Before actually sending, we check that
the service has the right permissions and that the metadata for the
letter confirms the letter is valid (because the query string can be
changed).
2019-09-12 09:54:36 +01:00
Katie Smith
7368245c9a Show letter preview once file is uploaded
This shows the sanitised letter preview if the file had no validation
errors or the preview with the overlay if it failed validation.
2019-09-12 09:54:36 +01:00
Katie Smith
8a322b844b Sanitise uploaded letters and store in S3
This sanitises uploaded letters and stores the sanitised result in S3
with if it passes validation or the original PDF in S3 if validation
fails. A metadata value of 'status' is set to either 'valid' or
'invalid'.
2019-09-12 09:54:36 +01:00
Katie Smith
5fa9e071c7 Add check that PDF file can be opened and is not malformed
This checks that the PDF file is not malformed in some way (e.g. by
missing the EOF marker). We check this by trying to get the page count
of the letter which will be needed to display the preview of the letter.
2019-09-12 09:54:36 +01:00
Katie Smith
a103dbf801 Add max file size check when uploading a letter 2019-09-12 09:54:36 +01:00
Katie Smith
be6b5b922a Add virus scan stage when uploading a letter 2019-09-12 09:54:36 +01:00
Katie Smith
c577416866 Add letter upload form which redirects to blank preview page
Added a form to upload a single letter. Currently this only uses the
form to validate that a file is submitted and that the file is a PDF. If
either of these validations fail, the form will display an error.
Otherwise, we redirect to a new preview page which just has the filename
as the heading for now.
2019-09-12 09:54:36 +01:00
Katie Smith
f04512a31d Bump utils 2019-09-12 09:54:35 +01:00
Chris Hill-Scott
47d5593f5f Merge pull request #3114 from alphagov/no-ie8
Remove code specific to IE8 and below
2019-09-11 11:14:27 +01:00
Chris Hill-Scott
3b159bac98 Merge pull request #3120 from alphagov/trust-self-define
Let NHS Trusts and clinical commissioning groups choose their own organisation before going live
2019-09-10 15:13:38 +01:00
Chris Hill-Scott
8b8893ed1d Let NHS Trusts and CCGs choose own organisation
All we do via support is ask which organisation they work for and
manually assign their service to it. This commit makes that process self
service.

We think we have all the trusts and clinical commissioning groups
loaded into the database now.

This will make the go live process smoother for these teams.
2019-09-06 16:26:51 +01:00
Chris Hill-Scott
d37566a79b Merge pull request #3117 from alphagov/gp-self-define
Let GP surgeries create their own organisations before going live
2019-09-06 16:24:25 +01:00
Chris Hill-Scott
fcb6f87d8b Make h1 consistent from page to page 2019-09-05 15:18:30 +01:00
Chris Hill-Scott
e0ab43f988 Let GPs nominate service name
Most GP practice services are named after the practice, which is the
organisation.

So rather than make people re-type the name of their organisation (and
potentially make a typo) let’s just let them say ‘yes, that’s the name
of my organisation’.
2019-09-05 15:01:12 +01:00
Chris Hill-Scott
daeefefeaa Let GP surgeries create their own organisations
We have a bunch of GP surgeries who want to go live. They don’t
automatically assigned to organisations. So this means a lot of back and
forth to get these organisations set up, and then the service has to
re-request to go live, and… it’s painful.

Instead, let’s let GPs create their own organisations, by confirming the
name of their organisation before going on to letting them accept the
agreement.
2019-09-05 15:01:12 +01:00
Chris Hill-Scott
d41effe8ce Allow GPs to click through to the agreement
We want GPs to be able to accept the agreement online. But at the moment
they don’t get automatically assigned to organisations. So we need to
let them enter the agreement accepting journey even if they don’t have
an organisation set up.
2019-09-05 14:46:02 +01:00
Tom Byers
c28d5632bf Merge pull request #3116 from alphagov/add-js-tests-for-prevent-duplicate-form-submissions.js
Add tests for prevent duplicate form submissions JS
2019-09-05 11:19:55 +01:00
Tom Byers
45e61d6ba5 Fix description of time until re-enable clicks 2019-09-05 10:52:13 +01:00
Tom Byers
9d8fcf34fc Fix variable assignment
Without the `var` prefix this was making
`$submitButton` a global variable, which is not
what the code intended.
2019-09-05 10:52:13 +01:00
Tom Byers
9967f29ca6 Add tests for preventing duplicate form submit 2019-09-05 10:52:13 +01:00
Tom Byers
3c1576dacf Add helper for mocking/stubbing form.submit
JSDOM doesn't implement the submit method on form
elements. It does have a submit method but this
just fires a 'not implemented' error.

We need to spy on form submissions fired by
clicking on the submit button but can't because
this event calls the submit method internally so
spying on `form.submit` doesn't work.

This adds a helper which spies on the internal
method that is actually called. When JSDOM
implements the submit method properly this should
be removed.
2019-09-05 10:52:13 +01:00
Chris Hill-Scott
df2742f797 Only serve Javascript to users of IE9 and above
Our Javascript errors in older versions of IE anyway, so we might as
well save people some bandwidth and page load speed by not serving it to
them.
2019-09-05 09:51:01 +01:00
Leo Hemsted
73587798ab Merge pull request #3115 from alphagov/utils-bump
utils bump
2019-09-04 11:50:24 +01:00
Leo Hemsted
6f99e55594 utils bump 2019-09-04 10:57:48 +01:00
Chris Hill-Scott
cc713b4057 Remove code specific to IE8 and below
Our usage for these browsers in the last month is down to 0.2% of all
users, or 14 individual users, according to Google Analytics.

These users also visit about half the number of pages per sessions,
suggesting that they’re not signed in.
2019-09-03 10:10:27 +01:00
Chris Hill-Scott
2fe7887ec6 Merge pull request #3113 from alphagov/remove-expanded
Update utils to remove toggle for expanding email previews
2019-09-03 09:15:28 +01:00
Chris Hill-Scott
1f2863a48d Merge pull request #3102 from alphagov/pyup-scheduled-update-2019-08-26
Scheduled weekly dependency update for week 34
2019-09-03 09:15:16 +01:00
Chris Hill-Scott
40e020d40d Remove expand_emails argument from get_template
It isn’t used for anything now.
2019-09-02 17:04:53 +01:00
Chris Hill-Scott
17b8c0194b Update utils and remove expanded argument
This argument was removed in https://github.com/alphagov/notifications-utils/pull/643
2019-09-02 17:04:53 +01:00
Leo Hemsted
0cf1251cad Merge pull request #3101 from alphagov/billing-for-all-services-report
Usage for all services
2019-09-02 16:38:23 +01:00
karlchillmaid
df617c570e Merge pull request #3112 from alphagov/send-a-file-formatting-update
Send a file by email – reintroduce instructions
2019-09-02 16:05:54 +01:00
karlchillmaid
f2611f61b8 Update app/templates/partials/templates/guidance-send-a-document.html
Co-Authored-By: Chris Hill-Scott <me@quis.cc>
2019-09-02 15:48:27 +01:00
karlchillmaid
a94ca9082c Remove comment-out
Because we've added 'Send a document' content to all client docs and we added channel-specific features pages with 'Send files by email' content, we're going to reintroduce the instructions that were commented out as part of https://github.com/alphagov/notifications-admin/pull/2270
2019-09-02 15:30:34 +01:00
Tom Byers
af689d4ac3 Merge pull request #3108 from alphagov/remove-expand-collapse-js
Remove expandCollapse JS
2019-08-30 14:29:34 +01:00
Chris Hill-Scott
c1014b1589 Merge pull request #3109 from alphagov/set-free-allowance-gp
Set free allowance for GP surgeries
2019-08-30 10:06:35 +01:00
Tom Byers
26e1e3a5cb Remove CSS tied to .js-enabled class
Not as obviously connected to expandCollapse but
no longer needed.
2019-08-30 08:44:14 +01:00
Chris Hill-Scott
13a98a73c1 Set free allowance for GP surgeries
This also tests that a user from an unknown organisation can pick any
of the available options and get the right allowance.
2019-08-29 17:47:14 +01:00
Tom Byers
c9c3f90fd8 Remove CSS used by expandCollapse.js 2019-08-29 16:42:47 +01:00
Tom Byers
030979a864 Remove expandCollapse.js script and refs to it 2019-08-29 16:42:27 +01:00
Sakis
5cd52d7996 Merge pull request #3107 from alphagov/bump-utils-again
Bump utils again
2019-08-29 16:33:07 +01:00
Athanasios Voutsadakis
f0011cbd58 Bump rest of the packages 2019-08-29 14:47:15 +01:00
Athanasios Voutsadakis
c35e2c1270 Bump utils to 33.2.9 2019-08-29 14:46:59 +01:00
Tom Byers
68d533fa18 Merge pull request #3103 from alphagov/split-up-js-test-helpers
Split JS test helpers into separate files
2019-08-29 14:02:25 +01:00