iobio / vcf.iobio.io

MIT License
27 stars 11 forks source link

Update for gru #67

Closed anderspitman closed 4 years ago

anderspitman commented 4 years ago

@tonydisera if you could spend 10-15 minutes reviewing this when you get a chance, I think that would be a good idea. No rush on that though. I've already deployed it to production, so it's not holding anything back at the moment.

The changes mostly have to do with local file support. I added a couple files for communicating with gru and fibridge (the local file proxy), and removed a bunch of files that were for handling VCFs in the browser. I'm pretty sure I didn't remove anything that's still needed, though there might still be a couple more we could get rid of. The RDP stuff for example only appears to be used in certain code paths that I'm not sure are ever exercised.

Most important thing will be testing the production instance to make sure everything is still working as expect. I'm make a slack message about that.