Skip to content

Connections: step-by-step guide only in the add/fix dialog - #54

Merged
joaoguimaraespro merged 1 commit into
mainfrom
feat/connections-guide-in-dialog
Oct 8, 2026
Merged

joaoguimaraespro merged 1 commit into
mainfrom
feat/connections-guide-in-dialog

Conversation

@joaoguimaraespro

Copy link
Copy Markdown
Owner

Summary

  • The page no longer shows the guide below the connections and again in the dialog. With no connections, the empty state offers one button per broker; each opens the form with that broker's guide expanded (first connection of that broker) and scrolled to step 1.
  • "How to fix it" on a failing connection opens its credentials form with the guide open, instead of scrolling to a page section.
  • Credential field labels/hints can be broker-specific (connections.fields.<Kind>.<key>): Binance's API Key / Secret Key no longer show Trading 212 hints.
  • The last guide step says to paste in the form below (the guide is now part of the form); Portuguese Binance guide uses "tu" like the rest of the app.

Test plan

  • ng build --configuration production, npm test (120 passed)
  • Screenshots with a mocked API: empty state, Binance dialog (guide at step 1, Binance field hints)
  • CI

🤖 Generated with Claude Code

- The page no longer repeats the guide: with no connections it shows one button per broker, and each opens
  the form with its guide expanded (first connection of that broker) and scrolled to step 1.
- 'How to fix it' on a failing connection opens its credentials form with the guide open.
- Field labels and hints can be broker-specific: Binance's API Key / Secret Key no longer show Trading 212 hints.
- Last guide step now says to paste below; Portuguese Binance guide uses 'tu' like the rest of the app.
@joaoguimaraespro
joaoguimaraespro merged commit 9d00e38 into main Oct 8, 2026
10 checks passed
@joaoguimaraespro
joaoguimaraespro deleted the feat/connections-guide-in-dialog branch October 8, 2026 20:06
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.

1 participant