-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fix: Findings regarding table refactoring #4047
Conversation
2fdbd9e
to
0a07e8e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix: Findings regarding table refactoring
0a07e8e
to
c736b32
Compare
Hey @iamsamgibbs thank you for your review 🙌 I fixed the issue with the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
Testing
TODO: Let's test these issues are fixed.
Note
Make sure to start a fresh environment using
node scripts/create-data.js --timeout 100 --coloniesCount 12
Manage tokens
Manage tokens
actionSplit payment
Step 1. Create a
Split payment
actionStep 2. Add more than 10 recipients
Step 3. Check pagination is not shown
Step 4. Switch to a mobile/tablet view
Step 5. Check
Percent
is shown for each rowStep 6. Check the footer amount and percent are right-aligned and have a right padding
Step 7. Complete the action and check steps
3-6
Edit colony
Edit colony details
actionResolves #4046