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.

Sign up for GitHub

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

Jump to bottom

Improve review load (WIP) #3189

Open
wants to merge 62 commits into
base: main
Choose a base branch
from
Open

Improve review load (WIP) #3189

wants to merge 62 commits into from
+916 −479

Conversation

fhennig
Copy link
Contributor

@fhennig fhennig commented Nov 7, 2024 •
edited
Loading

resolves #

preview URL: https://improve-review-load.loculus.org

Summary

Screenshot

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by an appropriate test.

fhennig and others added 30 commits October 30, 2024 11:09
@fhennig
Add a little comment
eabde75
@fhennig
initial refactoring of the view
bd0d824
@fhennig
Fixed all tests
a1250a8
@fhennig
linting
7e9ea3a
@fhennig
fix deprecation
25a5abc
@fhennig
fix imports
eec5a20
@fhennig
progress
a1b6fb3
@fhennig
rework filter
e2f2898
@fhennig
rework warnings and errors filters
5593812
@fhennig
test fixes
d2000c0
@fhennig
Add TODO
94f84ea
@fhennig
format
af26b43
@fhennig
Merge remote-tracking branch 'origin/main' into feat/3031-improve-rev…
2f7aa3f 
…iew-page
@fhennig
fix CI
2544de8
@fhennig
fix format
7d9e04a
@fhennig
remove dead code
4e8e77b
@fhennig
default null values
9850019
@fhennig
Improve documentation
589189a
@fhennig
Update API docs
7608bb1
@fhennig
improve error message
7603cc7
@fhennig
remove todo
d5d8e40
@fhennig
format
02415e0
@fhennig
remove log files
049b44f
@fhennig
rename perfect -> no_issues
54c8e27
@fhennig
Merge branch 'main' into feat/3031-improve-review-page
18fcbb2
@fhennig
some test improvements
1269166
@fhennig
remove outdated TODO
01a6a5d
@fhennig
Change wording
2a22284
@fhennig
rename isError and isWarning
6f7c757
@fhennig
improve processing result condition handling
59f7328
fhennig and others added 27 commits November 4, 2024 14:56
@fhennig
more fix stuff
019176b
@fhennig
Merge remote-tracking branch 'origin/main' into feat/3030-improve-rev…
302913a 
…iew-page
@fhennig
more fix stuff
00d2d07
@actions-user
Update schema documentation based on migration changes
9cb6cb8
@fhennig
more fix stuff
f2dd78a
@fhennig
refactor query
942774a
@fhennig
Merge branch 'main' into feat/3031-improve-review-page
1c3a9f7
@fhennig
Update docs
25bae87
@fhennig
Update docs
4634988
@fhennig
Show configuration dialog on any sequence action
ead805c
@fhennig
Improve confirm dialog
37a338c
@fhennig
test fix
49c98f5
@fhennig
fix e2e tests
00cc739
@fhennig
Add DB field for processing result
b2a7324
@fhennig
cleanup
fbd5320
@actions-user
Update schema documentation based on migration changes
241764e
@fhennig
bump
f59b6d0
@fhennig
un-bump
6fface7
@fhennig
frontend all fixed?
690f9ed
@fhennig
Merge branch 'main' into feat/3031-improve-review-page
eed85ff
@fhennig @fengelniederhammer
Update backend/src/main/resources/db/migration/V1.5__update_sequence_…
cb2296a 
…entries_view_status_field.sql

Co-authored-by: Fabian Engelniederhammer <[email protected]>
@fhennig
cleanup formatting
b7bc02c
@fhennig
simplify
487651e
@fhennig
remove TODO
6477e6d
@fhennig
Merge branch 'main' into feat/3031-improve-review-page
63a6b6c
@fhennig
Update diagrams
c205464
@fhennig
WIP - try CI
309ce2a
@fhennig fhennig added the preview Triggers a deployment to argocd label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
preview Triggers a deployment to argocd
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fhennig @actions-user