Skip to content

Conversation

mdrehan369
Copy link

Issue #66

Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for paisable ready!

Name Link
🔨 Latest commit 80e2904
🔍 Latest deploy log https://app.netlify.com/projects/paisable/deploys/68e8af3a473ee20008a9b661
😎 Deploy Preview https://deploy-preview-97--paisable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 95
Accessibility: 100
Best Practices: 100
SEO: 91
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@archa8 archa8 linked an issue Oct 8, 2025 that may be closed by this pull request
Copy link
Member

@archa8 archa8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly replace this const userId = new mongoose.Types.ObjectId("68e60a61599d061c090b8e61"); with the logic to fetch the userId automatically from the database. Also, you have made some unnecessary formatting changes like removing/adding spaces in transactionController.js. Kindly remove those and also remove the emojis from the log and error messages

@mdrehan369
Copy link
Author

Done!

Copy link
Member

@archa8 archa8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

The x-axis markers go outside the cards, which is not good for UI. Also, while hovering on the points, there is no clarity on what the numbers represent to a layman, which is not good for UX. You can fetch the currency as set by the user upon signup from the backend and then show the corresponding currency on the y-axis along with the amount, while clearly mentioning that the x-axis represents the date of transaction. Replace Activity Count on the legend as well as on hovering effect to expenditure, which will suit the UI/UX more.

Copy link
Member

@archa8 archa8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@archa8 archa8 merged commit 38edd4e into Code-A2Z:master Oct 10, 2025
4 checks passed
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.

feat: Implement Expenses vs Time and Income vs Time Charts

2 participants