Draft
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Wire the new apartment purchase feature into the app by adding its nav graph to HedvigNavHost, DI module to ApplicationModule, and navigation callback to InsuranceGraph. Also handle the new CrossSellInfoType.Purchase in the cross-sell sheet's exhaustive when expression. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Parses the storeUrl to detect apartment products (hyresratt/bostadsratt in the URL path) and routes them to the native purchase flow instead of opening the browser. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
HedvigScaffold already provides a scrollable Column, so adding verticalScroll inside it caused infinite height constraints. Also hardcode all cross-sell clicks to route to in-app purchase for testing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…croll Removes nested verticalScroll + weight(1f) that caused infinite height crash. Uses HedvigScaffold which handles scrolling internally. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Text state was managed in the Molecule presenter via events, causing a
round-trip delay on each keystroke that produced cursor jumping and
text disappearing. Now text fields use local remember { mutableStateOf }
in the composable and only send values to the presenter on submit.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…droid) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Without remember, state was recreated on every recomposition, preventing tier selection and navigation from working. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Detects if BankID app is installed via PackageManager. If installed, auto-opens the app (existing behavior). If not installed, shows a live QR code that can be scanned from another device. Uses the same ZXing-based QR rendering pattern as the login flow. The liveQrCodeData is already returned by the signing poll query and updates dynamically. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…orm and summary UI - Fix signing->success navigation getting stuck by using Unit as LaunchedEffect key instead of state (fires once when entering Success/Failed branch) - Group tier offers by tierDisplayName with deductible radio selector per tier, defaulting to lowest deductible - Add horizontal padding (16dp) to form content and subtitle text - Show strikethrough gross price on summary when discounted - Show deductible display in summary card - Fix HedvigButton missing required enabled parameter in QR code screen Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…tion Three fixes for the signing flow: 1. Poll immediately on first iteration instead of waiting 2s 2. Use FetchPolicy.NetworkOnly to skip Apollo cache (was returning stale QR code data and stale signing status) 3. Use a hasNavigated flag for success/failure navigation instead of LaunchedEffect(Unit) which could fail on recomposition Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The purchase graph is already popped when navigating to Success, so typedPopBackStack for the graph fails and falls through to finishApp(). Use plain popBackStack() instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
feature-purchase-apartmentmodule with form → tier selection → summary → BankID signing → success flowDetails
Test plan
🤖 Generated with Claude Code