Skip to content

feat: Add core services & navigation#1638

Open
bthebladeprimer wants to merge 2 commits into
bn/feature/cc-08-analytics-logging-a11yfrom
bn/feature/cc-09-services-navigation
Open

feat: Add core services & navigation#1638
bthebladeprimer wants to merge 2 commits into
bn/feature/cc-08-analytics-logging-a11yfrom
bn/feature/cc-09-services-navigation

Conversation

@bthebladeprimer
Copy link
Copy Markdown
Contributor

Summary

  • Add ConfigurationService for fetching and caching checkout configuration
  • Add ErrorMessageResolver for mapping validation errors to user-facing messages
  • Add navigation system: CheckoutCoordinator, CheckoutNavigator, BackportedNavigationStack, and CheckoutRoute
  • Add service protocols: RawDataManagerProtocol, VaultManagerProtocol, AnalyticsSessionConfigProviding
  • Add CheckoutComponentsStrings constants for localized string keys

Context

PR 9 of 16 in the CheckoutComponents feature split. Depends on PR 8 (analytics, logging & accessibility).

Notion tracker

Test plan

  • ConfigurationServiceTests — verify config fetch, caching, and error handling
  • ErrorMessageResolverTests / ErrorMessageResolverFieldErrorTests — verify error-to-message mapping
  • CheckoutCoordinatorTests — verify route coordination
  • CheckoutNavigatorTests — verify navigation stack management
  • AnalyticsSessionConfigProviderTests — verify session config generation

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 26, 2026

Warnings
⚠️ > Pull Request size seems relatively large. If this Pull Request contains multiple changes, please split each into separate PR will helps faster, easier review.

Generated by 🚫 Danger Swift against 04cf0f1

@bthebladeprimer bthebladeprimer marked this pull request as draft March 26, 2026 16:32
@bthebladeprimer bthebladeprimer force-pushed the bn/feature/cc-08-analytics-logging-a11y branch from 2945751 to c1eb1bb Compare March 31, 2026 14:07
@bthebladeprimer bthebladeprimer force-pushed the bn/feature/cc-09-services-navigation branch from 6d57497 to 0a8b7bc Compare March 31, 2026 14:07
@bthebladeprimer bthebladeprimer marked this pull request as ready for review April 3, 2026 18:09
@bthebladeprimer bthebladeprimer force-pushed the bn/feature/cc-08-analytics-logging-a11y branch from ef5c8c1 to af0c77e Compare April 15, 2026 13:02
@bthebladeprimer bthebladeprimer requested a review from a team as a code owner April 15, 2026 13:02
@bthebladeprimer bthebladeprimer force-pushed the bn/feature/cc-09-services-navigation branch from d0e13e1 to 04cf0f1 Compare April 15, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant