Skip to content

Toolbox on dashboard page should have relevant node tools and add tooltip text to back button#105

Merged
Taylor-DeVries merged 4 commits intomainfrom
toolbox-on-dashboard-should-have-relavent-node-tools-and-back-text-on-quiz-back-button
Jan 16, 2026
Merged

Toolbox on dashboard page should have relevant node tools and add tooltip text to back button#105
Taylor-DeVries merged 4 commits intomainfrom
toolbox-on-dashboard-should-have-relavent-node-tools-and-back-text-on-quiz-back-button

Conversation

@Aiden25451
Copy link
Copy Markdown
Collaborator

@Aiden25451 Aiden25451 commented Jan 16, 2026

Describe your changes

  • Extracted the logic to determine which calculators to show into a utility function
  • Used the new utility function to control what to show for both the dashboard and the quiz to ensure they are always the same
  • Updated the default case for tools on the dashboard to show only one link
  • Added tooltip text to the back button
  • Updated ESLint to stop screaming about having ' and " in our text. This is technically less safe, but if someone messed up, the build should fail. I don't think this should be the linter's responsibility.

Jira story link

Story Link 1
Story Link 2

Verification of changes

  • All tools showing on the quiz seem to match the dashboard. Still have different default cases though
  • It says back
image

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested my code with current main branch

…inks in default case to one link to learn wealthsimple since we talked about two links looking weird with same icon
@Aiden25451 Aiden25451 self-assigned this Jan 16, 2026
@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Jan 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
finbud Ready Ready Preview, Comment Jan 16, 2026 7:05pm

…an error this should be caught it build step anyways
@Aiden25451 Aiden25451 marked this pull request as ready for review January 16, 2026 19:06
@Taylor-DeVries Taylor-DeVries merged commit 21cf173 into main Jan 16, 2026
4 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