Update Barcode Scanner

[teravest]

Addresses issue #626

This updates the barcode scanner to be able to edit the fields in the barcode table.

I also created a controlled vocabulary for the sequencing_status column.

I also rewrote a giant req.write in barcode_utils.psp so that it was in html instead of a string.

[ElDeveloper]

• [x] Make a header for american gut vs the name of the project.
• [x] For biomass remaining change checkbox to drop down.
• [x] In the edit existing kit in the american gut utilities remove the project drop down.
• [x] Make project group table that determines where the project belongs to.
• [x] Add a column to project
• [x] Add a column to project_barcode

[teravest]

Ok all these changes should be made with the info we have right now.

[ElDeveloper]

:+1:

[teravest]

This is on webdev now :)

[adamrp]

Thanks!

On Fri, Feb 21, 2014 at 4:22 PM, teravest notifications@github.com wrote:

This is on webdev now :)

Reply to this email directly or view it on GitHubhttps://github.com/qiime/qiime_web_app/pull/632#issuecomment-35784374 .

[adamrp]

Minor aesthetic comment: can you change the <td> status rectangle (the thing that's green, red, purple, etc.) to have colspan=2 and align=center?

[adamrp]

metadata checker gets triggered regardless of project and/or project_group, so that should be conditional for AG at this point

[adamrp]

The default value for "Sequencing Status" in the database is NULL, so I think there needs to be an "unknown" (or "N/A" or something to that effect) option in that dropdown

[adamrp]

Many of the fields have a "Current: " next to them (I assume so that you can tell if you're changing something before you hit submit?), but others don't.

[adamrp]

Barcode 000007185 is office succession, but the checkbox or emailing says "Send kit owner an email" I might be mistaken, but I don't think that study has kits

[adamrp]

Not sure, but I think only projects in the AG project group currently work (do not have access to SQL developer atm)

[teravest]

I forgot the code to look up the barcode itself currently uses a stored procedure that looks in an american gut table. So this will only work with American Gut right now. I am going separate that into two stored procedures and only call the american gut one if it is an american gut barcode :)

[teravest]

Updates are deployed on Webdev Here is a list of barcodes to test with. One from each project 000000001 000006136 000005074 000007185 000005038 000007869 000010307 000011423 000006020

[teravest]

Updated webdev again this morning.

[ElDeveloper]

@teravest not sure if this has to be part of this pull request, but I looked for barcode 000004595 and sent an e-mail, the e-mail came in as:

   In order to help you in adding these details, the side of the sample tube 000004595 reads:
      sample date: [Field('sample_date', ''), Field('sample_date', '')] 
      sample time: [Field('sample_time', ''), Field('sample_time', '')]

I'm looking at the code now.

[ElDeveloper]

[ElDeveloper]

Great stuff @teravest

[adamrp]

@teravest, is this ready to be merged?

[teravest]

no, I have some things to change and its gonna need another review and more testing :(

On Wed, Mar 26, 2014 at 11:07 AM, adamrp notifications@github.com wrote:

@teravest https://github.com/teravest, is this ready to be merged?

Reply to this email directly or view it on GitHubhttps://github.com/qiime/qiime_web_app/pull/632#issuecomment-38710888 .

[ElDeveloper]

Fair enough, thanks @teravest