Skip to content

Commit bb327b7

Browse files
authored
Merge pull request #1656 from session-foundation/feat/pro-settings-qa
fix: pro settings qa
2 parents 4428768 + 7c13328 commit bb327b7

File tree

5 files changed

+37
-32
lines changed

5 files changed

+37
-32
lines changed

ts/components/dialog/debug/FeatureFlags.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -430,13 +430,13 @@ export const ProDebugSection = ({ forceUpdate }: { forceUpdate: () => void }) =>
430430
label="Expiry"
431431
flag="mockProAccessExpiry"
432432
options={[
433-
{ label: 'Soon', value: MockProAccessExpiryOptions.SOON },
434-
{ label: 'Today', value: MockProAccessExpiryOptions.TODAY },
435-
{ label: 'Tomorrow', value: MockProAccessExpiryOptions.TOMORROW },
436-
{ label: '1 Week', value: MockProAccessExpiryOptions.WEEK },
437-
{ label: '1 Month', value: MockProAccessExpiryOptions.MONTH },
438-
{ label: '3 Months', value: MockProAccessExpiryOptions.THREE_MONTH },
439-
{ label: '1 Year', value: MockProAccessExpiryOptions.YEAR },
433+
{ label: '7 Days', value: MockProAccessExpiryOptions.P7D },
434+
{ label: '29 Days', value: MockProAccessExpiryOptions.P29D },
435+
{ label: '30 Days', value: MockProAccessExpiryOptions.P30D },
436+
{ label: '30 Days 1 Second', value: MockProAccessExpiryOptions.P30DT1S },
437+
{ label: '90 Days', value: MockProAccessExpiryOptions.P90D },
438+
{ label: '300 Days', value: MockProAccessExpiryOptions.P300D },
439+
{ label: '365 Days', value: MockProAccessExpiryOptions.P365D },
440440
{ label: '24 Days 1 Minute', value: MockProAccessExpiryOptions.P24DT1M },
441441
{ label: '24 Hours 1 Minute', value: MockProAccessExpiryOptions.PT24H1M },
442442
{ label: '23 Hours 59 Minutes', value: MockProAccessExpiryOptions.PT23H59M },

ts/components/dialog/user-settings/pages/user-pro/ProNonOriginatingPage.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -284,6 +284,7 @@ function ProInfoBlockUpdate() {
284284
}
285285

286286
function ProInfoBlockRenew() {
287+
const dispatch = useDispatch();
287288
const { data } = useProAccessDetails();
288289
return (
289290
<ProInfoBlockLayout
@@ -296,6 +297,9 @@ function ProInfoBlockRenew() {
296297
icon={LUCIDE_ICONS_UNICODE.EXTERNAL_LINK_ICON}
297298
/>
298299
}
300+
descriptionOnClick={() =>
301+
showLinkVisitWarningDialog('https://getsession.org/pro-roadmap', dispatch)
302+
}
299303
subtitleElement={
300304
<ProInfoBlockSectionSubtitle>
301305
{tr('proOptionsTwoRenewalSubtitle')}
@@ -323,7 +327,7 @@ function ProInfoBlockCancel() {
323327
const { data } = useProAccessDetails();
324328
return (
325329
<ProInfoBlockLayout
326-
titleElement={tr('cancelAccess')}
330+
titleElement={tr('proCancellation')}
327331
descriptionElement={
328332
<Localizer
329333
token="proCancellationDescription"

ts/components/dialog/user-settings/pages/user-pro/ProSettingsPage.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -899,7 +899,7 @@ function PageHero() {
899899
dispatch(
900900
updateLocalizedPopupDialog({
901901
title: { token: 'proStatusError' },
902-
description: { token: 'proStatusNetworkErrorDescription' },
902+
description: { token: 'proStatusRefreshNetworkError' },
903903
overrideButtons: backendErrorButtons,
904904
})
905905
);

ts/hooks/useHasPro.ts

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -147,25 +147,25 @@ function proAccessOriginatingPlatformToStrings(
147147
}
148148

149149
function useMockProAccessExpiry() {
150-
const variant = useDataFeatureFlag('mockProAccessExpiry') ?? MockProAccessExpiryOptions.MONTH;
150+
const variant = useDataFeatureFlag('mockProAccessExpiry') ?? MockProAccessExpiryOptions.P30D;
151151

152-
// NOTE: for testing the expiry time should be pinned to x time after "now"
153-
const now = variant ? Date.now() : 0;
152+
// NOTE: for testing the expiry time should be pinned to x + 250ms after "now", the +250ms prevents render lag from changing the timestamp
153+
const now = variant ? Date.now() + 250 : 0;
154154
switch (variant) {
155-
case MockProAccessExpiryOptions.SOON:
156-
return now + 600 * 1000;
157-
case MockProAccessExpiryOptions.TODAY:
158-
return now + 12 * 60 * 60 * 1000;
159-
case MockProAccessExpiryOptions.TOMORROW:
160-
return now + 26 * 60 * 60 * 1000;
161-
case MockProAccessExpiryOptions.WEEK:
162-
return now + 8 * 24 * 60 * 60 * 1000;
163-
case MockProAccessExpiryOptions.MONTH:
155+
case MockProAccessExpiryOptions.P7D:
156+
return now + 7 * 24 * 60 * 60 * 1000;
157+
case MockProAccessExpiryOptions.P29D:
158+
return now + 29 * 24 * 60 * 60 * 1000;
159+
case MockProAccessExpiryOptions.P30D:
164160
return now + 30 * 24 * 60 * 60 * 1000;
165-
case MockProAccessExpiryOptions.THREE_MONTH:
161+
case MockProAccessExpiryOptions.P30DT1S:
162+
return now + 30 * 24 * 60 * 61 * 1000;
163+
case MockProAccessExpiryOptions.P90D:
166164
return now + 90 * 24 * 60 * 60 * 1000;
167-
case MockProAccessExpiryOptions.YEAR:
168-
return now + 12 * 30 * 24 * 60 * 60 * 1000;
165+
case MockProAccessExpiryOptions.P300D:
166+
return now + 300 * 24 * 60 * 60 * 1000;
167+
case MockProAccessExpiryOptions.P365D:
168+
return now + 365 * 24 * 60 * 60 * 1000;
169169
case MockProAccessExpiryOptions.P24DT1M:
170170
return now + 24 * 24 * 60 * 60 * 1000 + 60 * 60 * 1000;
171171
case MockProAccessExpiryOptions.PT24H1M:

ts/state/ducks/types/releasedFeaturesReduxTypes.ts

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,15 +29,16 @@ export type SessionFeatureFlags = {
2929
fsTTL30s: boolean;
3030
};
3131

32+
// ISO8601 duration format
3233
export enum MockProAccessExpiryOptions {
33-
SOON = 0,
34-
TODAY = 1,
35-
TOMORROW = 2,
36-
WEEK = 3,
37-
MONTH = 4,
38-
THREE_MONTH = 5,
39-
YEAR = 6,
40-
// The following are test cases from the PRD in ISO8601 duration format
34+
P7D = 0,
35+
P29D = 1,
36+
P30D = 2,
37+
P30DT1S = 3,
38+
P90D = 4,
39+
P300D = 5,
40+
P365D = 6,
41+
// The following are test cases from the PRD
4142
P24DT1M = 7,
4243
PT24H1M = 8,
4344
PT23H59M = 9,

0 commit comments

Comments
 (0)