Add shared budgets#67
Merged
Merged
Conversation
Contributor
Author
PR Workflow UpdatePreview URL: https://xpenser-pr-067.cleverbrush.com Workflow run / artifacts: https://github.com/cleverbrush/xpenser/actions/runs/28860710673 Manual preview QA completed in the deployed preview:
Validation:
SigNoz:
Note: I attempted to update the PR description with this status, but the GitHub integration returned |
Contributor
Author
|
Telegram PR-ready notification was skipped: |
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
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.
Original request
Plan and implement the new Budgets feature for xpenser:
MainbudgetLatest follow-up implemented in this update:
What changed
/:idroutes./blog/shared-budgetsinto a full SEO-friendly article targetingshared budget expense tracker, with internal links, richer copy, a hero image, and inline screenshots.next/image, plus a focused MDX component test.API batching note
API batching was not disabled globally. The only intentional direct/no-batch usage is on the budget detail management flow, where route-specific budget detail, member, invite, and currency reads/mutations need independent invalidation and clearer request behavior. The rest of the app keeps the existing API client behavior.
Blog update
/blog/shared-budgetsshared budget expense trackerapps/web/public/blog/shared-budgets-overview.pngapps/web/public/blog/shared-budget-detail.pngapps/web/public/blog/shared-budget-transactions.pngValidation
Local validation on
feat/budgets:npx vitest --run apps/web/components/mdx-components.test.tsx apps/web/lib/blog.test.ts apps/web/components/blog-pages.test.tsxnpm run lintnpm run typechecknpm testgit diff --checkPreview validation:
/blogand/blog/shared-budgetson desktop and mobile.GitHub validation:
92291d7PR Environments#271, run id28943291046, completed successfully.Telemetry check:
429from enrichment traffic and failed preview login attempts before using existing auth state.Notification:
127.0.0.1:8787was not reachable (curlconnection failure).