Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dni
Copy link
Member

@dni dni commented May 11, 2024

2024-05-11.11-21-14.mp4

refactor payments table and payment chart into components and add them to usermanager

refactor payments table and payment chart into components and add them
to usermanager
@codecov
Copy link

codecov bot commented May 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.66%. Comparing base (9f8942a) to head (c5993d9).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2491   +/-   ##
=======================================
  Coverage   61.65%   61.66%           
=======================================
  Files          68       68           
  Lines        9720     9721    +1     
=======================================
+ Hits         5993     5994    +1     
  Misses       3727     3727           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@motorina0
Copy link
Collaborator

nitpick: when the payments dialog is open the user has two dialogs one on top of the other, that is not great UX.

Copy link
Collaborator

@motorina0 motorina0 left a comment

Choose a reason for hiding this comment

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

tACK

@arcbtc
Copy link
Member

arcbtc commented May 13, 2024

Wow

@arcbtc arcbtc merged commit a5623ef into dev May 13, 2024
@arcbtc arcbtc deleted the payments-usermanager branch May 13, 2024 17:01
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.

4 participants