Skip to content

Add map download file size to travel advice download link [WHIT-2598] - #5109

Merged
GDSNewt merged 1 commit into
mainfrom
add-file-size-to-download-link
Nov 12, 2025
Merged

Add map download file size to travel advice download link [WHIT-2598]#5109
GDSNewt merged 1 commit into
mainfrom
add-file-size-to-download-link

Conversation

@GDSNewt

@GDSNewt GDSNewt commented Nov 3, 2025

Copy link
Copy Markdown
Contributor

What

  • Include the human-readable file size in the map download caption when present (e.g. "Download a more detailed map (PDF, 197 KB)").
  • Add unit tests for map_download_file_size.

Why

Makes download size visible to users so they can decide whether to download mobile-data-heavy files. This was identified as an issue in a DAC review.

Corresponding backend PR: alphagov/travel-advice-publisher#2394

Screenshot 2025-11-11 at 11 46 25 Screenshot 2025-11-11 at 11 47 05

Jira card

@govuk-ci
govuk-ci temporarily deployed to govuk-frontend-app-pr-5109 November 3, 2025 10:00 Inactive
@GDSNewt GDSNewt changed the title Add map download file size to travel advice download link Add map download file size to travel advice download link [WHIT - 2598] Nov 3, 2025
@GDSNewt
GDSNewt force-pushed the add-file-size-to-download-link branch from 2b8eb45 to ada2dec Compare November 3, 2025 10:58
@govuk-ci
govuk-ci temporarily deployed to govuk-frontend-app-pr-5109 November 3, 2025 10:58 Inactive

@ChrisBAshton ChrisBAshton left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but we should hold off on merging until the corresponding bit in TAP is implemented. (Also needs a review from the team that owns Frontend)

Comment thread app/views/travel_advice/_first_part.html.erb Outdated
@GDSNewt GDSNewt changed the title Add map download file size to travel advice download link [WHIT - 2598] Add map download file size to travel advice download link [WHIT-2598] Nov 4, 2025
Comment thread app/views/travel_advice/_first_part.html.erb Outdated
@GDSNewt
GDSNewt force-pushed the add-file-size-to-download-link branch from 51babe4 to 24414b8 Compare November 12, 2025 15:53
@govuk-ci
govuk-ci temporarily deployed to govuk-frontend-app-pr-5109 November 12, 2025 15:53 Inactive
@GDSNewt
GDSNewt force-pushed the add-file-size-to-download-link branch from 24414b8 to a7819b1 Compare November 12, 2025 15:58
@govuk-ci
govuk-ci temporarily deployed to govuk-frontend-app-pr-5109 November 12, 2025 15:58 Inactive
@GDSNewt
GDSNewt force-pushed the add-file-size-to-download-link branch from a7819b1 to 1f6acf7 Compare November 12, 2025 16:03
@govuk-ci
govuk-ci temporarily deployed to govuk-frontend-app-pr-5109 November 12, 2025 16:03 Inactive
@GDSNewt
GDSNewt merged commit b3b1b00 into main Nov 12, 2025
13 checks passed
@GDSNewt
GDSNewt deleted the add-file-size-to-download-link branch November 12, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants