Conversation
actual-github-bot
bot
changed the title
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
✅ Deploy Preview for actualbudget ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This comment has been minimized.
This comment has been minimized.
✅ Deploy Preview for actualbudget-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughSidebar label changed from "Reports" to "Reports Dashboard". The main reports page was rewritten toward a dashboard-centric structure: built-in reports converted into dashboard widgets, new images and headings added, navigation reordered, and an "Experimental Features" subsection introduced. (50 words) Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🤖 Auto-generated Release Notes Hey @youngcw! I've automatically created a release notes file based on CodeRabbit's analysis: Category: Enhancements If you're happy with this release note, you can add it to your pull request. If not, you'll need to add your own before a maintainer can review your change. |
✅ Deploy Preview for actualbudget-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
youngcw
changed the title
youngcw
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🤖 Fix all issues with AI agents
In `@packages/docs/docs-sidebar.js`:
- Line 188: The sidebar label currently reads 'Reports Dashboard' but the page
title in index.md is 'Reports Dashboards'; make them consistent by either
renaming the sidebar label string 'Reports Dashboard' to 'Reports Dashboards' or
updating the index.md title to 'Reports Dashboard'—ensure the label in
docs-sidebar.js and the title in index.md match exactly.
In `@packages/docs/docs/reports/index.md`:
- Line 8: Replace the incorrect contraction "it's" with the possessive "its" in
the sentence "If one dashboard is not enough, you can make as many dashboards as
you like, each with it's own set of reports and widgets." (look for that exact
sentence in packages/docs/docs/reports/index.md) so it reads "each with its own
set of reports and widgets."
- Line 15: Update the sentence "Currently, Actual comes with the following built
in widgets and reports:" to hyphenate the compound adjective by changing "built
in" to "built-in" (i.e., "Currently, Actual comes with the following built-in
widgets and reports:") so the phrase is grammatically correct; locate and edit
that exact string in the docs content for the file where it appears.
- Around line 36-41: In the "## Net Worth Graph" paragraph remove the duplicated
sentence "Net worth shows the balance of all accounts over time" so it only
appears once, and fix the grammar by changing "minus and debts you owe" to
"minus any debts you owe"; update the paragraph text under the heading to read
clearly (keep references to investments and the definition of net worth) so the
duplicate is gone and "and" is replaced with "any".
- Line 38: The Markdown image tag currently has no alt text
(); update this line to include a concise,
descriptive alt string that explains the image (for example "Net worth chart
showing assets and liabilities over time") so the image becomes accessible to
screen readers and static analysis passes; modify the alt text inside the square
brackets of the existing image reference in packages/docs/docs/reports/index.md.
- Line 23: The "[Custom Reports]" link currently points to the broken fragment
"#/docs/reports/custom-reports.md"; update that link target to a proper relative
page link (e.g., "custom-reports.md" or "./custom-reports.md" or the Docusaurus
route "/docs/reports/custom-reports" depending on site conventions) so the
"[Custom Reports]" anchor resolves correctly; locate the link text "[Custom
Reports]" in the markdown and replace the URL portion
"#/docs/reports/custom-reports.md" with the chosen valid relative path.
| { | ||
| type: 'category', | ||
| label: 'Reports', | ||
| label: 'Reports Dashboard', |
There was a problem hiding this comment.
Inconsistent naming: "Reports Dashboard" vs "Reports Dashboards".
The sidebar label here is "Reports Dashboard" (singular), but the page title in index.md line 1 is "Reports Dashboards" (plural). Pick one and use it consistently.
🤖 Prompt for AI Agents
In `@packages/docs/docs-sidebar.js` at line 188, The sidebar label currently reads
'Reports Dashboard' but the page title in index.md is 'Reports Dashboards'; make
them consistent by either renaming the sidebar label string 'Reports Dashboard'
to 'Reports Dashboards' or updating the index.md title to 'Reports
Dashboard'—ensure the label in docs-sidebar.js and the title in index.md match
exactly.
| ## Net Worth Graph | ||
|
|
||
|  | ||
|
|
||
| Net worth shows the balance of all accounts over time. Your net worth is all of your cash and assets minus and debts you owe. Net worth shows the balance of all accounts over time, including all of your investments. Your "net worth" is considered to be the amount you'd have if you sold all your assets and paid off as much debt as possible. | ||
|
|
There was a problem hiding this comment.
Duplicate sentence and grammar error in Net Worth description.
Line 40 has two issues:
- "minus and debts" should be "minus any debts."
- "Net worth shows the balance of all accounts over time" appears twice in the same paragraph.
-Net worth shows the balance of all accounts over time. Your net worth is all of your cash and assets minus and debts you owe. Net worth shows the balance of all accounts over time, including all of your investments. Your "net worth" is considered to be the amount you'd have if you sold all your assets and paid off as much debt as possible.
+Net worth shows the balance of all accounts over time, including all of your investments. Your net worth is all of your cash and assets minus any debts you owe. Your "net worth" is considered to be the amount you'd have if you sold all your assets and paid off as much debt as possible.🧰 Tools
🪛 LanguageTool
[grammar] ~40-~40: Ensure spelling is correct
Context: ...th is all of your cash and assets minus and debts you owe. Net worth shows the bala...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🪛 markdownlint-cli2 (0.20.0)
[warning] 38-38: Images should have alternate text (alt text)
(MD045, no-alt-text)
🤖 Prompt for AI Agents
In `@packages/docs/docs/reports/index.md` around lines 36 - 41, In the "## Net
Worth Graph" paragraph remove the duplicated sentence "Net worth shows the
balance of all accounts over time" so it only appears once, and fix the grammar
by changing "minus and debts you owe" to "minus any debts you owe"; update the
paragraph text under the heading to read clearly (keep references to investments
and the definition of net worth) so the duplicate is gone and "and" is replaced
with "any".
✅ Deploy Preview for actualbudget-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@packages/docs/docs/reports/index.md`:
- Line 13: Replace the empty alt text in the image markdown
"" with a concise, descriptive alt
string that conveys the content and purpose of the image (for example: "Reports
dashboard showing key metrics, charts, and filters"); update the line in
packages/docs/docs/reports/index.md so the image uses that descriptive alt text
to satisfy accessibility and markdownlint requirements.
- Line 32: The image markdown for /img/reports/cash-flow.webp currently has an
empty alt attribute; replace the empty brackets in the line
"" with a short, descriptive alt text (e.g.
"Cash flow chart showing monthly inflows and outflows") so the image has
meaningful accessibility text and satisfies markdownlint.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@packages/docs/docs/reports/index.md`:
- Line 52: The heading text "Use markdown to generate a text box to use as
report headings and organizational labels." should capitalize the proper noun
"Markdown"; update the string containing "Use markdown ..." (search for that
exact phrase) to "Use Markdown to generate a text box to use as report headings
and organizational labels." so the language name is correctly capitalized.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@packages/docs/docs/reports/index.md`:
- Around line 30-34: The "Cash Flow Graph" heading and the image alt text "Cash
flow Report" are inconsistent; update the image alt text in the markdown (the
alt string in the ![...]) to match the heading—e.g., change "Cash flow Report"
to "Cash Flow Graph" so the alt text and the "Cash Flow Graph" heading are
identical.
🧹 Nitpick comments (1)
packages/docs/docs/reports/index.md (1)
9-9: Consider simplifying "All of the" to "All the" for conciseness.This is a minor style suggestion to make the sentence more concise.
✏️ Proposed simplification
-All of the available reports can be tuned with filters and live or static date ranges. +All the available reports can be tuned with filters and live or static date ranges.
| ## Cash Flow Graph | ||
|
|
||
|  | ||
|  | ||
|
|
||
| Cash flow shows your spending over time. It only looks at budgeted accounts and shows their balance over time, and includes separate income and expense visualizations. It's a quick way to see how your income and expenses affected your available money over time. Your budgeted accounts are considered to be "cash on hand," so this gives you a picture of how available money fluctuates. |
There was a problem hiding this comment.
Inconsistency: heading says "Graph" but image alt text says "Report".
The section heading is "Cash Flow Graph" but the image alt text says "Cash flow Report". These should match for consistency.
📝 Proposed fix for consistency
-
+🤖 Prompt for AI Agents
In `@packages/docs/docs/reports/index.md` around lines 30 - 34, The "Cash Flow
Graph" heading and the image alt text "Cash flow Report" are inconsistent;
update the image alt text in the markdown (the alt string in the ![...]) to
match the heading—e.g., change "Cash flow Report" to "Cash Flow Graph" so the
alt text and the "Cash Flow Graph" heading are identical.
1 participant
Getting the basics ready for better reports documentation.