Skip to content

Commit 4aa1a73

Browse files
Show a success message after requesting a new download
1 parent 7a6829a commit 4aa1a73

5 files changed

Lines changed: 10 additions & 10 deletions

File tree

app/controllers/download.js

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,8 @@ export const downloadController = {
103103
create(request, response) {
104104
const { account } = request.app.locals
105105
const { programmeType, session_id, type } = request.body.download
106-
const { data } = request.session
106+
const { data, referrer } = request.session
107+
const { __ } = response.locals
107108

108109
let createdDownload
109110
if (type) {
@@ -132,13 +133,9 @@ export const downloadController = {
132133

133134
const download = new Download(createdDownload, data)
134135

135-
// Generate and return file
136-
const { buffer, fileName, mimetype } = download.createFile(data)
136+
request.flash('success', __(`download.new.success`, { download }))
137137

138-
response.header('Content-Type', mimetype)
139-
response.header('Content-disposition', `attachment; filename=${fileName}`)
140-
141-
response.end(buffer)
138+
response.redirect(referrer)
142139
},
143140

144141
download(request, response) {

app/locales/en.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -724,7 +724,9 @@ export const en = {
724724
'{count, plural, =0 {No exports matching your search criteria were found} one {Showing <b>{from}</b> to <b>{to}</b> of <b>{count}</b> export} other {Showing <b>{from}</b> to <b>{to}</b> of <b>{count}</b> exports}}',
725725
new: {
726726
label: 'Download vaccination report',
727-
confirm: 'Download vaccination data'
727+
confirm: 'Download vaccination data',
728+
success:
729+
'The records will take some time to download. You will be able to download them soon in [Exports](/downloads)'
728730
},
729731
createdAt: {
730732
label: 'Requested at'

app/views/download/form.njk

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
{% extends "_layouts/form.njk" %}
22

3+
{% set formAction = "/downloads/new?referrer=/reports" %}
34
{% set confirmButtonText = __("download.new.confirm") %}
45
{% set title = __("download.new.label") %}
56

app/views/move/list.njk

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
value: DownloadType.Moves,
2222
decorate: "download.type",
2323
attributes: {
24-
formaction: "/downloads/new",
24+
formaction: "/downloads/new?referrer=/moves",
2525
formmethod: "post"
2626
}
2727
}) }}

app/views/session/offline.njk

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
{% set title = __("session.offline.title") %}
44
{% set confirmButtonText = __("session.offline.confirm") %}
5-
{% set formAction = "/downloads/new" %}
5+
{% set formAction = "/downloads/new?referrer=" + session.uri %}
66

77
{% block beforeContent %}
88
{{ breadcrumb({

0 commit comments

Comments
 (0)