-
-
Notifications
You must be signed in to change notification settings - Fork 159
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 for Issue #1013 #1070
Fix for Issue #1013 #1070
Conversation
The problem with tests for previous code base is at Transfer.update() |
I think hooks need to be setup like it was done for Task to store update history. |
tests pass for node v17.3.0 |
Great @webbdays with the solution, I just changed the name of the component to make it more concise, but great job and well done with the build fix! 🎉 |
Thank you. |
But tests for new functionality in frontend are not covered. And also code docs. |
@webbdays I will have to revert this PR, still an issue on the logged page: Please check it and create a new PR with your fixes. |
ok. |
Pull Request Template
Description
To allow user to sort the columns in a table
Purpose
To allow user to sort the columns in a table
Solution
Using TableSortLabel component from MUI, Got to know about it after seeing the PR's related to this issue.
Screenshots
None
Checklist
Related Issues
None
Additional Notes
None
Contributor Guidelines
Fork the repo and create your branch from master. - done
If you've added code that should be tested, add tests. - not yet
If you've changed APIs, update the documentation. - NA
Ensure the test suite passes. - tests on previous codebase are failing
Make sure your code lints. - linting on previous codebase are failing
Issue that pull request! - going to do it
Check if the build is green before we start to review - waiting
Thank you.