UserOfficeProject / issue-tracker

Shared place for features and bugs from all collaborators.
0 stars 0 forks source link

Scientists should be able to change statuses on Xpress proposals #1138

Closed simonfernandes closed 2 days ago

simonfernandes commented 3 months ago

Scientists should be able to select from a list of static statuses for proposals within their technique.

So far these are the statuses that we have talked about, but this list still needs to be finalised:

From: Gozzard, Emma (STFC,RAL,ISIS) emma.gozzard@stfc.ac.uk Sent: Thursday, July 25, 2024 1:08 PM To: Fernandes, Simon (STFC,RAL,ISIS) simon.fernandes@stfc.ac.uk Cc: Singh, Sukhvinder (STFC,RAL,ISIS) Sukhvinder.Singh@stfc.ac.uk Subject: RE: Xpress questions

I think, at the very least, we need:

Submitted Under review Unsuccessful or Reject Queued or Approved (i.e. approved and waiting to be run) Finished

simonfernandes commented 1 month ago

Hi @Cat-Lucifer. You previously mentioned these as a rough idea for proposal statuses. I've since been looking at the legacy Xpress import and we'll need to map the statuses over (I suppose ideally we might want to just mark all old proposals as rejected or finished, but I don't think there's a way of knowing which ones are actually still "in progress", is there?). Does this mapping help to come up with a final status list, and how does the mapping look to you? Legacy is on the left and new on the right side.

Submitted (locked) Under review Unsuccessful Approved Finished

  ['No status', 'Submitted (locked)'],
  ['Submitted', 'Submitted (locked)'],
  ['Saved', DO NOT IMPORT'],
  ['Sample size and/or hazards review', 'Under review'],
  ['Under Review', 'Under review'],
  ['Under Review- with can', 'Under review'],
  ['Unsuccessful', 'Unsuccessful'],
  ['Sample being returned to user', 'Approved'],
  ['Awaiting Can', 'Approved'],
  ['Awaiting Sample', 'Approved'],
  ['No sample can- awaiting sample', 'Approved'],
  ['Successful- request can', 'Approved'],
  ['Successful- send sample', 'Approved'],
  ['Queued', 'Approved'],
  ['Disposed of by ISIS', 'Approved'],
  ['Successful', 'Approved'],
  ['Sample size and/or hazards accepted- please send sample', 'Approved'],
  ['Finished', 'Finished'],
  ['Closed', 'Finished'],
simonfernandes commented 1 month ago

Hi @Cat-Lucifer, following on from our discussion, please could you check this? Thanks 😊

Status/instrument changes:

Status mappings:

If the proposal is submitted before 01/01/2024:

Drafts are not imported, everything is finished except unsuccessful.

If the proposal is submitted after 01/01/2024:

  ['No status', 'Submitted (locked)'],
  ['Submitted', 'Submitted (locked)'],
  ['Saved', DO NOT IMPORT'],
  ['Sample size and/or hazards review', 'Under review'],
  ['Under Review', 'Under review'],
  ['Under Review- with can', 'Under review'],
  ['Unsuccessful', 'Unsuccessful'],
  ['Sample being returned to user', 'Approved'],
  ['Awaiting Can', 'Approved'],
  ['Awaiting Sample', 'Approved'],
  ['No sample can- awaiting sample', 'Approved'],
  ['Successful- request can', 'Approved'],
  ['Successful- send sample', 'Approved'],
  ['Queued', 'Approved'],
  ['Disposed of by ISIS', 'Approved'],
  ['Successful', 'Approved'],
  ['Sample size and/or hazards accepted- please send sample', 'Approved'],
  ['Finished', 'Finished'],
  ['Closed', 'Finished'],
Cat-Lucifer commented 1 month ago

@simonfernandes Thanks! I think that's broadly right.

For: "When the status is Approved or Finished, the scientist can change the instrument, but it won't have an effect downstream" this should be:

Thanks!

ETA - bold statues changed:

['No status', 'Submitted (locked)'], ['Submitted', 'Submitted (locked)'], ['Saved', DO NOT IMPORT'], ['Sample size and/or hazards review', 'Under review'], ['Under Review', 'Under review'], ['Under Review- with can', 'Under review'], ['Unsuccessful', 'Unsuccessful'], ['Sample being returned to user', 'Finished'], ['Awaiting Can', 'Approved'], ['Awaiting Sample', 'Approved'], ['No sample can- awaiting sample', 'Approved'], ['Successful- request can', 'Approved'], ['Successful- send sample', 'Approved'], ['Queued', 'Approved'], ['Disposed of by ISIS', 'Finished'], ['Successful', 'Finished'], ['Sample size and/or hazards accepted- please send sample', 'Approved'], ['Finished', 'Finished'], ['Closed', 'Finished'],

simonfernandes commented 3 weeks ago

Hi @Cat-Lucifer, I think I've got most of this logic down now and I'm hoping there might be time to show you a quick demo on Friday, but just in case:

Cat-Lucifer commented 3 weeks ago

@simonfernandes Responses for each below. Thanks!

  1. Sounds good to me.
  2. Yep, I think this is a good idea. I assume anybody with User Officer role can edit a proposal in these statuses if necessary?
  3. Not really. It would be good for these to have the instrument selected shown.
  4. Yes, happy with greyed out statuses
simonfernandes commented 1 week ago

@Cat-Lucifer Sorry, just realised I didn't answer your question!

  1. Yes, the way it's been done is so that scientists are restricted on what they can do based on the above conditions, but user officers can always change the instrument and status.
  2. I will make sure they're mapped over to an instrument then.