puhep / pudb

Purdue CMS FPix Database
0 stars 0 forks source link

Additional module location option of "In Transit" #127

Closed lantone closed 8 years ago

lantone commented 8 years ago

Hi @gneeser , I was just discussing with John and Doug here, we agree it'd be useful to add an option of "In Transit" for the "Current Location" of a module. At the moment rely on comments on an individual module page, but being able to view all the modules in transit via the Tested Modules List would help us organize our shifting manpower a couple of days in advance. It seems straightforward from where I sit (easy for me to say), but do you foresee any complications with adding another location option?

Thanks,

Jamie

gneeser commented 8 years ago

Jamie, It will take a little doing, but I don't foresee any problems as of now. I'll keep you posted if anything comes up.

-Greg

lantone commented 8 years ago

another (arguably better) option: just a boolean for "In Transit?", set to true when shipped and back to false when received. That way the "Current Location" would be free to be used to denote the destination of modules when in transit.

gneeser commented 8 years ago

Jamie, I added a yes/no filter option for in transit. Does this match what you were looking for?

-Greg

lantone commented 8 years ago

yeah, that's perfect. but now how do i mark a module as being in transit?

gneeser commented 8 years ago

Promoting it to shipped takes care of that.

lantone commented 8 years ago

but what if it's shipped multiple times? like a module that goes from UNL->KU->FNAL? once's it's "shipped" once can it be somehow promoted again?

gneeser commented 8 years ago

That's a good point, I hadn't thought of that. I added a checkbox on the Update Module Location page for 'In transit.' Just check that and update the location to the destination; when it arrives, you can update the location as normal.

lantone commented 8 years ago

Awesome! Thanks Greg!

gneeser commented 8 years ago

My pleasure!