kashish/expense-tracker-assessment
1
0
Fork 1
Code Issues Pull requests 1 Projects Releases Wiki Activity Actions

Expense Tracker assessment — Amanjot Singh #1

Open
amanjotsingh-jungleworks wants to merge 1 commit from amanjotsingh-jungleworks/expense-tracker-assessment:main into main
pull from: amanjotsingh-jungleworks/expense-tracker-assessment:main
merge into: kashish:main
Conversation 0 Commits 1 Files changed 22 +1010 -85
amanjotsingh-jungleworks commented 2026-06-08 10:18:43 +00:00
First-time contributor
Copy link

Completed implementation of the expense tracker assessment.

Architecture: a single signal-based store (ExpenseStore) holds all raw
state; the filtered list, totals, and breakdowns are computed signals
derived from it, so the list and summary stay in sync automatically.
Components are thin — they only read from or write to the store.

Highlights:

  • Reactive form with a custom pure-function validator (noFutureDate)
  • Route resolver pre-fetches the expense before the edit form loads
  • Field errors show on touch or submit; API failures surface inline
  • RelativeTimePipe with natural thresholds

Setup and run instructions are in README.md; design notes and decisions
are in NOTES.md / Architecture.md.

Completed implementation of the expense tracker assessment. Architecture: a single signal-based store (ExpenseStore) holds all raw state; the filtered list, totals, and breakdowns are computed signals derived from it, so the list and summary stay in sync automatically. Components are thin — they only read from or write to the store. Highlights: - Reactive form with a custom pure-function validator (noFutureDate) - Route resolver pre-fetches the expense before the edit form loads - Field errors show on touch or submit; API failures surface inline - RelativeTimePipe with natural thresholds Setup and run instructions are in README.md; design notes and decisions are in NOTES.md / Architecture.md.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u main:amanjotsingh-jungleworks-main
git switch amanjotsingh-jungleworks-main

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff amanjotsingh-jungleworks-main
git switch amanjotsingh-jungleworks-main
git rebase main
git switch main
git merge --ff-only amanjotsingh-jungleworks-main
git switch amanjotsingh-jungleworks-main
git rebase main
git switch main
git merge --no-ff amanjotsingh-jungleworks-main
git switch main
git merge --squash amanjotsingh-jungleworks-main
git switch main
git merge --ff-only amanjotsingh-jungleworks-main
git switch main
git merge amanjotsingh-jungleworks-main
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
kashish/expense-tracker-assessment!1
No description provided.