Skip to content

Conversation

glad-adyen
Copy link
Contributor

@glad-adyen glad-adyen commented Jan 16, 2025

Summary

This PR introduces utilities for a more streamlined and memoized getBalanceAccounts functionality for active sessions.

  • getActiveSessionBalanceAccounts()

    Gets fresh balance accounts for the active session from the server (when possible).

  • getActiveSessionBalanceAccountsFactory()

    Returns a shared function for getting balance accounts for the active session with memoization (when applicable). Useful for fetching balance accounts only once per active session.

  • setupBalanceAccountsTest()

    Utility for setting the stage for session-aware tests that involve getting balance accounts from the server.

Fixed issue: CXP-2809: Fetch balance accounts utils

@glad-adyen glad-adyen self-assigned this Jan 16, 2025
@glad-adyen glad-adyen changed the title Utilities for get balance accounts Draft: Utilities for get balance accounts Jan 16, 2025
@glad-adyen glad-adyen changed the title Draft: Utilities for get balance accounts [DRAFT] Utilities for get balance accounts Jan 16, 2025
@glad-adyen glad-adyen changed the title [DRAFT] Utilities for get balance accounts Utilities for get balance accounts [DRAFT] Jan 16, 2025
# Conflicts:
#	src/core/Auth/session/__testing__/MockSessionTestUtils.ts
@glad-adyen glad-adyen marked this pull request as ready for review January 20, 2025 12:48
@glad-adyen glad-adyen changed the title Utilities for get balance accounts [DRAFT] Utilities for get balance accounts Jan 20, 2025
@glad-adyen glad-adyen force-pushed the feature/get-balance-accounts branch from 887ab51 to c462d24 Compare January 20, 2025 17:05
@glad-adyen glad-adyen changed the title Utilities for get balance accounts Get balance accounts utils Jan 21, 2025
german664
german664 previously approved these changes Jan 21, 2025
# Conflicts:
#	src/hooks/useBalanceAccounts/useBalanceAccounts.ts
@glad-adyen glad-adyen force-pushed the feature/get-balance-accounts branch from fdfb1cb to 9144f06 Compare January 27, 2025 14:46
Copy link
Contributor

github-actions bot commented Apr 7, 2025

This pull request has been marked as stale because it has been inactive for more than 14 days.
Please update this pull request or it will be automatically closed in 7 days.

@github-actions github-actions bot added the stale label Apr 7, 2025
@glad-adyen glad-adyen requested a review from a team as a code owner April 7, 2025 08:21
@github-actions github-actions bot removed the stale label Apr 14, 2025
Copy link
Contributor

github-actions bot commented Aug 4, 2025

This pull request has been marked as stale because it has been inactive for more than 14 days.
Please update this pull request or it will be automatically closed in 7 days.

@github-actions github-actions bot added the stale label Aug 4, 2025
@github-actions github-actions bot removed the stale label Aug 11, 2025
Copy link
Contributor

This pull request has been marked as stale because it has been inactive for more than 14 days.
Please update this pull request or it will be automatically closed in 7 days.

@github-actions github-actions bot added the stale label Oct 13, 2025
@glad-adyen glad-adyen closed this Oct 17, 2025
@glad-adyen glad-adyen removed the stale label Oct 17, 2025
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.

2 participants