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

Beta 2 More Feedback Improvements and Tests #62

Merged
merged 12 commits into from
Aug 16, 2022
Merged

Conversation

matthew-a-dunlap
Copy link
Contributor

No description provided.

Use single db query and python loops to get all users to populate who is on what manuscript
Switch manuscripts page name
Switch filter code to now search all columns for username as multi column searching is ugly in datatables
Makes it more obvious what is going on

Also, disable user filter table as it is unused and unweildy
Does not change across pages / searches
Did not set submission or user table
Fix file datatable error when not showing newness column
Access assertions are disabled

Also:
Correct clean up selenium instances during tearDown
Fix ! point color
Minor button improvements for testing
We test admin functionality in our other tests already
This switch uncovered test parts that were using the curator admin when they should have been other roles
Fix incorrect test invites
Fix submission file datatable being visible to non authors when author is working on it

TODO:
- Add complete tests for previous submissions
- Manually test the file_table change and improve
- Rerun old tests, probably broken from above as well
Fix tests related to file_table
Fix minor issues other tests
Rename some previous submission test arrays
@matthew-a-dunlap matthew-a-dunlap merged commit 9530ebe into master Aug 16, 2022
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.

1 participant