Skip to content

Commit b08c7fd

Browse files
pglombardoPeter Giacomo Lombardo
andauthored
Clarify share message expiration copy. (#4657)
Content is deleted on expiration whether or not it was retrieved. Fixes #4485 Co-authored-by: Peter Giacomo Lombardo <pglombardo@apnotic.com>
1 parent a0562a3 commit b08c7fd

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

app/helpers/share_message_helper.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ def push_share_message_text(push, secret_url:, locale: nil)
1111
notes = [
1212
share_message_expiration_note(push),
1313
(passphrase_share_note if push.passphrase.present?),
14-
I18n._("Once retrieved, the link and content will be deleted upon expiration.")
14+
I18n._("The link and content will be deleted upon expiration.")
1515
].compact
1616

1717
[

test/helpers/share_message_helper_test.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@ class ShareMessageHelperTest < ActionView::TestCase
1818
assert_includes message, "IMPORTANT NOTES:"
1919
assert_includes message, @push.views_remaining.to_s
2020
assert_includes message, @push.days_remaining.to_s
21+
assert_includes message, "The link and content will be deleted upon expiration."
22+
assert_not_includes message, "Once retrieved"
2123
end
2224

2325
test "push_share_message_text includes passphrase note when passphrase set" do

0 commit comments

Comments
 (0)