Skip to content

Staging spark merged - #4341

Merged
thisconnect merged 3 commits into
BitBoxSwiss:staging-sparkfrom
thisconnect:staging-spark-merged
Aug 20, 2026
Merged

Staging spark merged#4341
thisconnect merged 3 commits into
BitBoxSwiss:staging-sparkfrom
thisconnect:staging-spark-merged

Conversation

@thisconnect

@thisconnect thisconnect commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Conflicts:
CHANGELOG.md
frontends/web/src/app.tsx
frontends/web/src/components/bottom-navigation/bottom-navigation.tsx
frontends/web/src/components/bottom-navigation/utils.test.ts
frontends/web/src/components/bottom-navigation/utils.ts
frontends/web/src/routes/router.tsx

After moving Insurance to the Marketplace, "Settings" was now the
only item inside "More", causing unnecessary friction to enter the
actual settings on mobile.

This moves settings directly into the menu bar, removing the redundant
intermediate step.
@thisconnect

Copy link
Copy Markdown
Collaborator Author

I've seem to have successfully merged, but didn't test everything

@thisconnect

Copy link
Copy Markdown
Collaborator Author

@sutterseba if you have a moment could oyu check that everything in 5ec120f works here?

@sutterseba sutterseba left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

there is a regression breaking the active tab indication now, fixed by sutterseba@fe0707b

@thisconnect
thisconnect force-pushed the staging-spark-merged branch 2 times, most recently from aafef49 to fe0707b Compare August 20, 2026 12:09
Conflicts:
	CHANGELOG.md
	frontends/web/src/app.tsx
	frontends/web/src/components/bottom-navigation/bottom-navigation.tsx
	frontends/web/src/components/bottom-navigation/utils.test.ts
	frontends/web/src/components/bottom-navigation/utils.ts
	frontends/web/src/routes/router.tsx
@thisconnect
thisconnect force-pushed the staging-spark-merged branch from fe0707b to 2470778 Compare August 20, 2026 12:17
@thisconnect
thisconnect marked this pull request as ready for review August 20, 2026 12:40
@thisconnect
thisconnect requested a review from a team as a code owner August 20, 2026 12:40

@sutterseba sutterseba left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks LGTM

@thisconnect
thisconnect merged commit 33c883d into BitBoxSwiss:staging-spark Aug 20, 2026
11 checks passed
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