Merged
Conversation
#254) Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no>
* Only genrequest members can mark an order as seen * Move logic closer to frontend * Removed unnecessary code, checks buttons everywhere, added check in post. * Removed prints * Fixed linter error
…Preparing for filters.
* Update IsolationMethod model to allow non-unique names and optimize queries in SampleLabView * Add macOS specific files to .gitignore * Add analysis orders, project, and isolation method fields to SampleCSVSerializer and update CSV export logic * Add marked, plucked, and isolated fields to SampleCSVSerializer and update SampleViewset for CSV export --------- Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no>
* Implement custom header rendering for sortable tables and enable multi-column sorting * Refactor sample status management Replace SampleStatusAssignment with boolean fields for marked, plucked, and isolated. Update other parts of the code which were effected by this change. --------- Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no>
* Implement custom header rendering for sortable tables and enable multi-column sorting * Refactor sample status management Replace SampleStatusAssignment with boolean fields for marked, plucked, and isolated. Update other parts of the code which were effected by this change. * Update titles for new order tables on dashboard * Add nowrap class to status rendering for better text display --------- Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no>
* Patch to sort properly when name is a mix of int and char, as well as only int or char. Generate genlab ID based on other fields (type). * Order is now only decided by the rows in the table, not a sorting_order value. * Update src/staff/tables.py Co-authored-by: Ole Magnus <me@omfj.no> * Added order.id to lock the db * added safe "select_for_update()" * Converted back to list and modified the tests to the current genlab id generation * Now abe to sort all types of names --------- Co-authored-by: Ole Magnus <me@omfj.no>
* Patch to sort properly when name is a mix of int and char, as well as only int or char. Generate genlab ID based on other fields (type). * Order is now only decided by the rows in the table, not a sorting_order value. * Update src/staff/tables.py Co-authored-by: Ole Magnus <me@omfj.no> * added safe "select_for_update()" * Removed prints --------- Co-authored-by: Ole Magnus <me@omfj.no>
Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no>
Set 'is_seen' to False when converted to draft. Order must be marked as "seen" to be shown in assigned orders (my orders). Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no>
Contributor
nicokant
approved these changes
Jul 21, 2025
Collaborator
nicokant
left a comment
nicokant
left a comment
There was a problem hiding this comment.
Everything seems ok, the only note is to keep the styling consistent, using as much as possible daisy-ui and tailwind classes even in the CSS.
This is not blocking, and it can be improved in the next weekly PR
github-merge-queue bot
pushed a commit
that referenced
this pull request
Jul 21, 2025
* Only one button for generating genlab id and set as "processing" (#248) * Add logic to mark order as completed when all samples are isolated (#250) Updated checkbox-name so that it is connected to the order-id Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> * Add priority to urgent orders table (#251) * Exclude 'pop_id' and 'location' fields from OrderExtractionSampleTable (#254) Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> * Only genrequest members can mark an order as seen (#252) * Only genrequest members can mark an order as seen * Move logic closer to frontend * Removed unnecessary code, checks buttons everywhere, added check in post. * Removed prints * Fixed linter error * Count isolated samples (#260) * Assign staff to all orders (#263) * Changes the buttons to a specified format. Moved buttons to the top. Preparing for filters. * Use correct delivery date on dashboard (#268) * Filter for orders and samples. Lab is missing. (#270) * Run CI on summer25-* * Download CSV of samples (#265) * Update IsolationMethod model to allow non-unique names and optimize queries in SampleLabView * Add macOS specific files to .gitignore * Add analysis orders, project, and isolation method fields to SampleCSVSerializer and update CSV export logic * Add marked, plucked, and isolated fields to SampleCSVSerializer and update SampleViewset for CSV export --------- Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> * Changed to tailwind styling (#273) * Completed order does not show in my orders and urgent orders (#275) * Add multicolumn sorting samples (#276) * Implement custom header rendering for sortable tables and enable multi-column sorting * Refactor sample status management Replace SampleStatusAssignment with boolean fields for marked, plucked, and isolated. Update other parts of the code which were effected by this change. --------- Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> * Change text on dashboard (#280) * Implement custom header rendering for sortable tables and enable multi-column sorting * Refactor sample status management Replace SampleStatusAssignment with boolean fields for marked, plucked, and isolated. Update other parts of the code which were effected by this change. * Update titles for new order tables on dashboard * Add nowrap class to status rendering for better text display --------- Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> * Fix name sorting (#281) * Patch to sort properly when name is a mix of int and char, as well as only int or char. Generate genlab ID based on other fields (type). * Order is now only decided by the rows in the table, not a sorting_order value. * Update src/staff/tables.py Co-authored-by: Ole Magnus <me@omfj.no> * Added order.id to lock the db * added safe "select_for_update()" * Converted back to list and modified the tests to the current genlab id generation * Now abe to sort all types of names --------- Co-authored-by: Ole Magnus <me@omfj.no> * Genlab ID patch (#271) * Patch to sort properly when name is a mix of int and char, as well as only int or char. Generate genlab ID based on other fields (type). * Order is now only decided by the rows in the table, not a sorting_order value. * Update src/staff/tables.py Co-authored-by: Ole Magnus <me@omfj.no> * added safe "select_for_update()" * Removed prints --------- Co-authored-by: Ole Magnus <me@omfj.no> * Sample list does no longer show guid or plate position, but shows location (#290) * Added filtering on types for extraction order (#287) * Maked sample statuses constants instead of strings (#289) Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> * Update order-status-logic when converted to draft (#284) Set 'is_seen' to False when converted to draft. Order must be marked as "seen" to be shown in assigned orders (my orders). Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> --------- Co-authored-by: Bertine <112892518+aastabk@users.noreply.github.com> Co-authored-by: Morten Lyngstad <81157760+mortenlyn@users.noreply.github.com> Co-authored-by: Morten Madsen Lyngstad <morten.lyngstad@bekk.no> Co-authored-by: aastabk <aastabk@stud.ntnu.no>
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.