BrickTwo / BrickHunter

26 stars 8 forks source link

Unable to transfer to LEGO website / Uncaught promise #204

Closed henriquedesousa closed 11 months ago

henriquedesousa commented 1 year ago

BrickHunter 2.0, latest Chrome Version 119.0.6045.106 (Official Build) (64-bit)

No messages in the Console tab of DevTools.

Notice how Pieces and Price appear as NaN. Country and language configured as such also on Lego website.

image

I also got another bug when importing the XML file mentioned in https://github.com/BrickTwo/BrickHunter/issues/184

image

Thanks in advance.

BrickTwo commented 1 year ago

HI,

Can you provide me your wantedList where you get the NaN error? Attach it here or send it to info@bricktwo.net, thx.

The second error is something else. it's currently not possible to import just one part, as soon as you add a second part (i.e. same part but different color), it's working. it's a know bug, but with low priority (who is just importing one part? :) ).

henriquedesousa commented 1 year ago

Done. Btw, website is https://www.lego.com/en-pt and settings are image

BrickTwo commented 1 year ago

Looks like the quantity is somehow not imported. Will try to fix it this weekend. As a workaround you can just assign a quantity for each position of the Parts List, after that, everything should work.

mevitsbricks commented 1 year ago

I believe I am seeing the same issue. I have quantities in my XML that weren't imported to BH. You mentioned assigning quantities for each position, is that something I can do via BH?

mevitsbricks commented 1 year ago

I had created the XML file I used to upload to BH via Brickstore. The brickstore xml export imported to BH without qty. I then tested uploading from brickstore to bricklink wanted list, then downloaded that wanted list from bricklink, and then imported it to BH which worked fine. The qty showed up. Just wondering how @henriquedesousa might have created the XML file.

henriquedesousa commented 1 year ago

I exported in BrickStore via "BrickLink XML" Ctrl+E, Ctrl+X

mevitsbricks commented 1 year ago

So both were done the same way. So something about the XML from brickstore and BL wanted lists are different.

On Fri, Nov 10, 2023 at 5:31 PM Henrique de Sousa @.***> wrote:

I exported in BrickStore via "BrickLink XML" Ctrl+E, Ctrl+X

— Reply to this email directly, view it on GitHub https://github.com/BrickTwo/BrickHunter/issues/204#issuecomment-1806559715, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV4DV7PE7UXOIHIKO6AAEWLYD22NBAVCNFSM6AAAAAA626FZDCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBWGU2TSNZRGU . You are receiving this because you commented.Message ID: @.***>

BrickTwo commented 11 months ago

Sorry for my late reply, I where in vacation far away in the last weeks :)

Seems like BrickStore is not using the same name for the Quantity (QTY instead on MINQTY) and Price (PRICE instead of MAXPRICE) information. That's why BrickHunter is not importing these information. Will try to support it in BrickHunter by this Weekend and publish it together with some Bug fixes.

image

BrickTwo commented 11 months ago

Fixed with Version 2.2.8

henriquedesousa commented 11 months ago

Thank you, will try as soon as I recover from my Hexenschuss :) how was your vacation, did you go very far?

mevitsbricks commented 11 months ago

Thank you very much!

Kevin (Mevit) www.mevitsbricks.com Brick by Brick By Brick!!!

On Sat, Dec 2, 2023 at 4:57 AM BrickTwo @.***> wrote:

Closed #204 https://github.com/BrickTwo/BrickHunter/issues/204 as completed.

— Reply to this email directly, view it on GitHub https://github.com/BrickTwo/BrickHunter/issues/204#event-11128780964, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV4DV7J434ORA6EEFNTRP23YHMCSXAVCNFSM6AAAAAA626FZDCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGEZDQNZYGA4TMNA . You are receiving this because you commented.Message ID: @.***>