Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invoice Dashboard Improvement #93

Merged
merged 6 commits into from
Mar 11, 2025
Merged

Conversation

HazelYuAhiru
Copy link
Contributor

@HazelYuAhiru HazelYuAhiru commented Feb 25, 2025

Description

Improved the Invoice Dashboard that shows all the invoices to match the new design.

Screenshots

Screen Shot 2025-03-06 at 12 58 07 PM

Issues

Closes #87

@HazelYuAhiru HazelYuAhiru linked an issue Feb 25, 2025 that may be closed by this pull request
4 tasks
Copy link
Collaborator

@jessieh9 jessieh9 left a comment

Choose a reason for hiding this comment

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

Hi @HazelYuAhiru @dnarimot , great work so far. Just a couple small fixes

  • For the table Headers <Th></Th>, can we try get this to match the figma in camel case and not all caps. Chakra UI by default keeps it all caps, but there is an attribute textTransform="none" that can prevent that. You might need to tweak font size a little bit
  • Same small fix for the "Past Due", "Not Paid" tags, change them from all caps to match design's camelcase

For a larger fix not previously mentioned,

  • I noticed that the invoices repeat themselves if they have different payers.

    • For example, the program, Testing class, with deadline 12/31/2024 appears twice on invoice page.
    • A program has invoice for each month and I believe what's happening is because they have multiple payers, it appears more than once.
  • The fix is to combine the returned elements if they have multiple payers. So the program, Testing program, with deadline in 12/31/2024 appearing twice would be combined; similar for the one with deadline in 02/20/2025

image

If you need extra time to implement this fix, feel free to let me know. Thanks.

@dnarimot dnarimot requested a review from jessieh9 March 10, 2025 09:31
@jessieh9 jessieh9 merged commit 65fb7f9 into main Mar 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invoice Dashboard Improvement
3 participants