On Wed, Nov 29, 2017 at 6:32 PM, Marc Juul <marc(a)juul.io> wrote:
Integration is definitely doable. The problem is that
the previous
scancode_decode.js (the one we currently use) was very incomplete and could
not interpret the characters that inform about errors. In order to detect
the errors the new scancode_decode.js must be used (from the commit I
previously mentioned) but using that scancode_decode means breaking
compatibility. The solution is to use the improved scancode_decode first and
if there are no errors but none of the hashes in the database matched then
retry with the old scancode_decode.
Ok, I just finally looked through the old and new code to integrate
them. I pushed a new branch called "errors" which I think will work.
I'll be at Omni tomorrow to test it and hopefully put it into
production.
https://github.com/sudoroom/doorjam/commit/d80d363b7e31fbb83a08226d08e733aa…