Repository navigation
Conversation
- Add docs/user-manual.md: user-facing manual covering the feature list, how to manage personal finances, and how to configure and use Folio - README.md: link the new documentation set and replace the inaccurate Supported Banks list with the verified Bank Connect connectors - docs/getting-started.md: cross-link the user manual from Next Steps
sensible-analytic
previously approved these changes
Oct 17, 2026
- ai-assistant: migrate useAssistantApi->useAui, useAssistantState->useAuiState (@assistant-ui/react 0.15.22 on main removed the deprecated aliases; PR merge refs inherit main's lockfile, so PR CI type-check failed on the merge ref even though this branch pins 0.12.24 where the aliases still exist) - market-data: allow clippy::double_must_use on MarketDataProvider async_trait; use sort_by_key instead of sort_by(..cmp..) (clippy::unnecessary_sort_by) - codeql: drop python from the analysis matrix (no python sources; job failed during python extraction)
- crate-level allow for clippy::double_must_use (203 errors generated by async_trait on returned futures, same pattern as the market-data fix) - sort_by_key/sort_by_key+Reverse instead of sort_by(..cmp..) (7x unnecessary_sort_by) - drop redundant borrows in format args (4x useless_borrows_in_formatting) - drop needless closure borrow (1x needless_borrows_for_generic_args) - drop redundant into_iter() before stream::iter (1x useless_conversion)
main's lockfile bumped prettier 3.8.1 -> 3.9.9, which reformats ~180 files; the PR Check workflow's format:check gate failed on all of them.
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
3 participants
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
Adds a user-facing manual for Sensible Folio and corrects documentation inaccuracies.
Changes
docs/user-manual.md— a complete user manual covering:Accuracy
The bank/broker list and import formats in the manual are grounded in source:
Verification
npm run lint→ 0 errors (1509 style warnings, tolerated)npm run build→tsc && vite buildsucceedsDocs-only change; no runtime code touched.
CI fixes (3bf7c11)
This PR also resolves pre-existing CI failures that surfaced on the PR merge ref (main + branch):
lint-and-test): migrateduseAssistantApi/useAssistantStatetouseAui/useAuiStatein the ai-assistant components. Main upgraded@assistant-ui/reactto 0.15.22, which removed those deprecated aliases; PR merge refs inherit main's lockfile, so CI failed even though this branch pins 0.12.24.#[allow(clippy::double_must_use)]onMarketDataProviderandsort_by_keyin the market-data providers.pythonfrom the analysis matrix (no Python sources; the job failed during Python extraction).