Skip to content

Conversation

@tembo
Copy link
Contributor

@tembo tembo bot commented Sep 7, 2025

Description

Add icon field to goals, allowing users to personalize goal tracking with emojis

Changes

  • Created Goal model with icon field
  • Added CRUD operations for goals
  • Implemented goal-related API endpoints
  • Created database migration for goal table
  • Added relationships with User, Account, Income, and Transfer models

Want me to make any changes? Add a review or comment with @tembo and i'll get back to work!

tembo.io

tembo bot and others added 8 commits September 5, 2025 21:52
- Add validation to ensure both from_account and to_account are found for transfers
- Add validation to prevent transfers between the same account
- Implement specific error messages for transfer validation failures
- Update WhatsApp parser to validate transfer requirements before processing

This addresses the requirement that BOTH accounts must be found and can't be
the same account to make a transfer via WhatsApp.
…e-implementation

WhatsApp Transfer Feature
- Added icon field to Goal model and schema for emoji support
- Created database migration for goal table with icon field
- Updated user and account relationships to include goals
- Added goal relationships to income and transfer models
- Integrated goals API endpoints into main router
- Icon field is optional/nullable to maintain backward compatibility

The icon field allows users to add emoji icons to their financial goals for better visualization and personalization.
@tembo tembo bot requested a review from EverStarck September 7, 2025 19:30
@tembo
Copy link
Contributor Author

tembo bot commented Sep 7, 2025

Requesting review from @EverStarck who has experience with the following files modified in this PR:

  • backend/app/app/db/base.py
  • backend/app/app/models/user.py
  • backend/app/app/models/income.py
  • backend/app/app/api/api_v1/api.py
  • backend/app/app/models/account.py
  • backend/app/app/models/transfer.py

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