OCR på räkningar

Diskussion i 'Appförslag och idéer' startad av Buzz, 21 jul 2010.

  1. pakerfeldt

    pakerfeldt Adult Droid Medlem

    Blev medlem:
    28 feb 2010
    Inlägg:
    716
    Mottagna gillanden:
    72

    MINA ENHETER

    Mycket bra input! Det betyder att min naiva approach inte höll ;)
    Om någon vet vart man får tag i mer uppgifter om dokumenttyperna så vore jag evigt tacksam. Verkar som att fler än 41 syftar till BG, uppenbarligen 42 också. Kanske flera?
     
  2. wulax

    wulax Kid Droid Medlem

    Blev medlem:
    7 jul 2010
    Inlägg:
    61
    Mottagna gillanden:
    7

    MINA ENHETER

    RickardP:
    Testa senaste versionen på GitHub. Jag har fixat NullPointerException tror jag. Kan inte upprepa FC:et i alla fall.
     
  3. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    ok, ändrade manuellt i filen lokalt och fungerar nu fint med OCR och beloppet att betala, ser inte ut att vara så vårt att implementera konto nummer också eller är det mer än i chrome extensionen?


    Om någon kan förklara hur jag updaterar projektet med eGit så :-) försökt hitta vart man kör update eller vad det nu heter på git men hittar inget, lyckades importera genom det men updatera?
     
  4. wulax

    wulax Kid Droid Medlem

    Blev medlem:
    7 jul 2010
    Inlägg:
    61
    Mottagna gillanden:
    7

    MINA ENHETER

  5. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Gjorde updatering av både android och chrome extensionen och fungerar fint med Nordea för OCR och betalnings belopp, av vilken anledning är inte konto fixat för Nordea? är det allmänt problem eller bara för Nordea, såg ut i js filen att någon annan bank hade det så borde inte vara så svårt om allt annat än js filen är fixad?
     
  6. johannilsson

    johannilsson Adult Droid Medlem

    Blev medlem:
    23 jun 2009
    Inlägg:
    577
    Mottagna gillanden:
    8

    MINA ENHETER

    Vet inte hur Nordea sidan ser ut än, ska kolla på dina filer senare. Om de ser bra ut fixar vi in stödet såvida du inte hinner före ;)
     
  7. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Fixat, dock inte så kunnig på git så du får koden här:

    Kod:
    function handleNordea(invoice, tab) {
        chrome.tabs.executeScript(tab.id, {
            code: "if ('"+ invoice.reference +"' != '') document.getElementById('paymentmessage').value= '"+ invoice.reference +"';" +
                  "if ('"+ invoice.amount +"' != '') document.getElementById('paymentamount').value= '"+ invoice.amount +"';" +
                  "if ('"+ invoice.account +"' != '') document.getElementById('paymenttoaccount').value= '"+ invoice.account +"'"
        });
    }
    Fungerar för mig lokalt.
     
  8. johannilsson

    johannilsson Adult Droid Medlem

    Blev medlem:
    23 jun 2009
    Inlägg:
    577
    Mottagna gillanden:
    8

    MINA ENHETER

    Perfekt tackar. Lägger in vid tillfälle.
     
  9. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Låt mig testa github så kommer jag med patch på rätt sätt :) kan vara roligt att testat
     
  10. Kaj

    Kaj Senior Droid Medlem

    Blev medlem:
    12 jun 2009
    Inlägg:
    1 768
    Mottagna gillanden:
    44

    MINA ENHETER

    Någon som har SEB som bank?
     
  11. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Så där ja en pull-request skickad ;-) säg gärna om jag verkar gjort något fel, vill gärna lära mig git och github men inte haft något bra projekt för det tidigare.
     
  12. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Vilken typ av betalnings avi man scannar skickas alldrig med till chrome extensionet va?

    Vore ju kanske bra att ha med ett värde i invoice objektet för det annars :-)

    Vore inte så svårt att få den att sätta rätt betalning i interfacet då, Nordea har 2st radio knappar för Bankgiro eller plusgiro.

    Edit: när jag ändå var i gång så kollade jag och hittade invoice.type så nu har jag commitat in en liten till fix för detta och ser ut att finnas med i pull requesten (antar den tar från jag synkade senast eller något?)
     
    Last edited: 24 jan 2011
  13. pakerfeldt

    pakerfeldt Adult Droid Medlem

    Blev medlem:
    28 feb 2010
    Inlägg:
    716
    Mottagna gillanden:
    72

    MINA ENHETER

    Huruvida det är PG eller BG borde vi absolut få över till webbläsaren. Idag skickar vi dokumenttypen från android -> servern men det är synd att behöva ha en uppfattning om vad dokumenttypen står för på flera ställen i droidgiro. Det borde räcka med android-appen.

    Johan, kan jag skicka "PG" eller "BG" som type till molnet istället för en siffra som idag? Isåfall kan vi åstadkomma detta med en enbart en fix i android-appen.
     
  14. pakerfeldt

    pakerfeldt Adult Droid Medlem

    Blev medlem:
    28 feb 2010
    Inlägg:
    716
    Mottagna gillanden:
    72

    MINA ENHETER

    Ser att du redan gjort en work-around. Jag vill helst skona webbläsar-pluggen från att behöva if-checka dokumenttypen på det där sättet och istället enbart kolla om det är BG eller PG.
     
  15. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Låter mycket bättre, men hur löser vi nu när jag redan commitat och pull requesten?

    Inte bättre acceptera den och sedan ändra i hela projektet?
     
  16. johannilsson

    johannilsson Adult Droid Medlem

    Blev medlem:
    23 jun 2009
    Inlägg:
    577
    Mottagna gillanden:
    8

    MINA ENHETER

    Låter mycket bättre, kör på.

    Edit, det var till @pakerfeldt ang BG och PG som dokumenttyp.
     
  17. johannilsson

    johannilsson Adult Droid Medlem

    Blev medlem:
    23 jun 2009
    Inlägg:
    577
    Mottagna gillanden:
    8

    MINA ENHETER

    Om du har baserat din pull request från master så kan du commita och pusha in fixen så får vi den senaste.

    Annars så dra ut en ny från vårt repo, brancha, pusha din branch och gör en pull request på den. Det här är det sättet som vi helst vill ha det på för att slippa efterföljande commits.
     
  18. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    Det lät dock som parkerfeldt ville fixa det där med BG och PG, därför tänkte jag bättre eftersom jag redan fixat commit och pull-request på mina ändringar så att ni accepterar dem sedan ändrar han hela kedjan.

    Men har ni bättre idè så...
     
  19. pakerfeldt

    pakerfeldt Adult Droid Medlem

    Blev medlem:
    28 feb 2010
    Inlägg:
    716
    Mottagna gillanden:
    72

    MINA ENHETER

    Jag ändrar så att jag skickar PG / BG istället för 41/42/etc som dokumenttyp från android -> molnet. Sen hur det hanteras i webbläsaren lägger jag mig inte i (än ;))
     
  20. RickardP

    RickardP Adult Droid Medlem

    Blev medlem:
    8 apr 2010
    Inlägg:
    515
    Mottagna gillanden:
    22

    MINA ENHETER

    okej kan fixa det jag har commitat så det stämmer med det, det blir alltså type = 'PG' eller 'BG' du skickar då eller?
     
    ef98dpa gillar detta.