Skip to content

Conversation

@isabellaenriquez
Copy link
Member

@isabellaenriquez isabellaenriquez commented May 13, 2025

Dependent on:

This information will be used for things such as product trial management, subscription usage cards, and other frontend rendering.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 13, 2025
@codecov
Copy link

codecov bot commented May 14, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
10302 1 10301 9
View the top 1 failed test(s) by shortest run time
useReplayCount getOne & hasOne should return 0 if the data is loaded but does not include a count for a requested id
Stack Traces | 0.195s run time
Error: expect(received).toBe(expected) // Object.is equality

Expected: 0
Received: undefined
    at Object.<anonymous> (.../utils/replayCount/useReplayCount.spec.tsx:85:45)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at listOnTimeout (node:internal/timers:555:9)
    at processTimers (node:internal/timers:529:7)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Copy link
Member Author

Choose a reason for hiding this comment

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

the diff looks larger than the actual changes i made (linting made it look like i changed a whole chunk when it was really just indented)

'seer_autofix',
'seer_scanner',
].includes(category)
['profile_duration', 'replays', 'spans', 'profile_duration_ui'].includes(category)
Copy link
Member Author

Choose a reason for hiding this comment

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

will make sure to update product trial prompts in product trial frontend pr that will make use of availableReservedBudgetTypes and canProductTrial

*/
totalReservedSpend: number;
};
} & ReservedBudgetCategory;
Copy link
Member Author

Choose a reason for hiding this comment

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

// @ts-expect-error TS(7053): Element implicitly has an 'any' type because expre... Remove this comment to see the full error message
formData.reserved[category] > 0 && (
{activePlan.categories
.filter(category => activePlan.planCategories[category])
Copy link
Member Author

Choose a reason for hiding this comment

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

we will render overview info for reserved budgets in checkout separately

@isabellaenriquez isabellaenriquez marked this pull request as ready for review May 14, 2025 19:07
@isabellaenriquez isabellaenriquez requested a review from a team as a code owner May 14, 2025 19:07
Comment on lines +80 to +81
seerAutofix: {singular: 'issue fix', plural: 'issue fixes'},
seerScanner: {singular: 'issue scan', plural: 'issue scans'},
Copy link
Member

@brendanhsentry brendanhsentry May 15, 2025

Choose a reason for hiding this comment

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

whoops i think the names should actually be "issue fix run" and "issue scan run". or we could change that in the backend. We can revisit in a future PR. I don't think the names are super finalized

@brendanhsentry brendanhsentry merged commit 68316ef into master May 15, 2025
41 checks passed
@brendanhsentry brendanhsentry deleted the isabella/reserved-budget-types branch May 15, 2025 16:33
@github-actions github-actions bot locked and limited conversation to collaborators May 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants