Added Empty State for Transaction List
Fix for #247
Hi, the PR doesn't actually reflect the figma file, you added in the homepage the widget that should show up only in the transactions page. See image below.
| Figma | PR |
|---|---|
Okay my bad I will do the required changes
@theperu I have completed the changes you can review & Merge now.
@uveshm003 thanks we'll take a look at this in the next days
@uveshm003 we have some high priority PRs that need to be merged first and then we will come back to you and the others. Unfortunately with the high activity of the project it's not easy to check everything in a short time
@uveshm003 I didn't test the PR yet, but just by looking at the code, I can tell that u didn't make the changes that luca pointed here. I can see that you're still showing up the calculator images that is not as the Figma was intended to be.
So please request a review only and only if you made the changes requested
Hey, The calculator image is only displayed in transactions screen not on home screen. I have used a Boolean variable to manage this. You can check the same.
Hey, The calculator image is only displayed in transactions screen not on home screen. I have used a Boolean variable to manage this. You can check the same.
You're right, my bad! I'll test it locally and give my feedback, sorry!
Okay
Hi @uveshm003👋 We recently merged a PR that changed the folder structure, which caused some conflicts here. When you get a chance, could you take a look and resolve them?
If there’s no response in the next 15 days, we’ll consider this PR inactive and plan to close it to keep things tidy.
Thanks!