Skip to content

Commit 5eaead2

Browse files
committed
Remove redundant comments in simplified subscription dialog code
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SE1tavFh7qJRq7hfMimbVa
1 parent b357dbb commit 5eaead2

2 files changed

Lines changed: 0 additions & 9 deletions

File tree

newIDE/app/src/Profile/Subscription/SubscriptionDialog/SimplifiedSubscriptionDialog.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -436,8 +436,6 @@ export default function SimplifiedSubscriptionDialog({
436436
</span>
437437
</Text>
438438
<Text noMargin size="section-title">
439-
{/* "Upgrade to […]" is intentionally kept as a client-side
440-
translation, with the plan name served by the backend. */}
441439
<Trans>Upgrade to {featuredPlanName}</Trans>
442440
</Text>
443441
</Column>

newIDE/app/src/Utils/GDevelopServices/Usage.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -191,8 +191,6 @@ export type Limits = {|
191191
purchasableQuantities: UsagePurchasableQuantities,
192192
},
193193
message: string | typeof undefined,
194-
// A/B test config for the subscription dialog, served with the limits.
195-
// Absent on older backends.
196194
subscriptionDialogDisplayConfig?: SubscriptionDialogDisplayConfig,
197195
|};
198196

@@ -222,9 +220,6 @@ export type SubscriptionPlanPricingSystem = {|
222220
periodCount: number,
223221
|};
224222

225-
// Store badges that can be shown next to a simplified bullet point. Kept as a
226-
// loose string so unknown (future) badges sent by the backend are simply
227-
// ignored by older clients.
228223
export type SimplifiedSubscriptionStoreBadge = string;
229224

230225
export type SimplifiedSubscriptionBulletPoint = {|
@@ -246,8 +241,6 @@ export type SubscriptionPlan = {|
246241
descriptionByLocale: MessageByLocale,
247242
bulletPointsByLocale: Array<MessageByLocale>,
248243
specificRequirementByLocale?: MessageByLocale,
249-
// Optional content for the simplified subscription dialog. Served by the
250-
// backend so it can be translated. Absent on older backends.
251244
simplifiedFeatures?: SimplifiedSubscriptionFeatures,
252245
targetAudiences: Array<'CASUAL' | 'PRO' | 'EDUCATION'>,
253246
fullFeatures: Array<{|

0 commit comments

Comments
 (0)