-
Notifications
You must be signed in to change notification settings - Fork 60
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
Test/pagination super admin #1288
Test/pagination super admin #1288
Conversation
@kevkevinpal , I'm having an issue with rendering MyTable and its css components. So I mocked the css using an empty file. Still, there is an error in HTML syntax. If you have any suggestions on this I would appreciate that. I'm working on it for now. |
…kmunde/sphinx-tribes into test/pagination-superAdmin
You have some jest and prettier tests failing. Can you check them plz? |
Yes I'm working on it. There are quite few changes and html syntax errors to be addressed. |
Hey @ecurrencyhodler , can I get a code review? It has a lot of changes and reverts too but I managed to add tests in |
All tests passed. GJ with that. Yes let's get a code review. |
@kevkevinpal , any other discrepancies found? Or changes needed to be made. @ecurrencyhodler , resolved the changes. |
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.
Nice Work!
paid |
* #1236 Added Organization Users unit test * Made changes for the mockFetch * made prettier changes * syntax and minor changes * #1233 Added test to check pagination. * Organization user conflict changes * removed mocked css line in PaginationView test * mock css added again * Ignore CSS Imports in Jest Configuration * added Alt text for pagination arrows * separated two <h2> styled elements. * mock data update * date format updated * <h2> child error changes * added mock bounties * changes * test changes * removed unnecessary files * removed excessive mock data * removed unncessary spaces --------- Co-authored-by: kevkevin <oapallikunnel@gmail.com>
* #1236 Added Organization Users unit test * Made changes for the mockFetch * made prettier changes * syntax and minor changes * #1233 Added test to check pagination. * Organization user conflict changes * removed mocked css line in PaginationView test * mock css added again * Ignore CSS Imports in Jest Configuration * added Alt text for pagination arrows * separated two <h2> styled elements. * mock data update * date format updated * <h2> child error changes * added mock bounties * changes * test changes * removed unnecessary files * removed excessive mock data * removed unncessary spaces --------- Co-authored-by: kevkevin <oapallikunnel@gmail.com>
Describe your changes
Issue ticket number and link
Type of change
Please delete options that are not relevant.
Checklist before requesting a review