search

zarmstrong / hlo-importer

A module for Foundry VTT that converts and imports Herolab Online 2e characters.
MIT License
6 stars 8 forks source link

Unable to import character #7

Closed StevenVanDyk closed 3 years ago

StevenVanDyk commented 3 years ago

Started a while ago I have a Cleric level 7 in hlo I generate the code and use the token to try to import. it comes back with the log to confirm.

the Character imported into cannot open. I tried with all other modules turned off incase of an incompatibility and in a new world

zarmstrong commented 3 years ago

Can you provide the token please? I'll need that to dig in.

StevenVanDyk commented 3 years ago

$pDaWYEQ~@P2#

Abs thanks

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 1:55 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

Can you provide the token please? I'll need that to dig in.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828658411, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDFDH4S4FS6FEB55ZKTTLBDXJANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

Yep, just noticed it on the other project :) Thanks

zarmstrong commented 3 years ago

It seems to be the ~ character. Can you hit the button in Hero Lab that says Revoke Element Token, then get a new element token?

zarmstrong commented 3 years ago

Actually, ignore that. That doesn't seem to help. I'll keep digging

StevenVanDyk commented 3 years ago

$QD8VR8o~@P2#

I tried a few here is the latest They all came up with the ~

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:04 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

It seems to be the ~ character. Can you hit the button in Hero Lab that says Revoke Element Token, then get a new element token?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828664519, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDCHSRHLHW6OOLLU6EDTLBEX7ANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

Yeah, I noticed that too. I'm digging in deeper, thanks for the updated token.

zarmstrong commented 3 years ago

I'm honestly thinking there's an issue with their API. I am opening a ticket with them.

StevenVanDyk commented 3 years ago

ok thanks. Much appreciate the Help and the Module

zarmstrong commented 3 years ago

No problem. They are somewhat slow to reply, so I'll update when I hear back. Going to try some back channels to see if I can get a faster response

zarmstrong commented 3 years ago

Ok, it's working now. Can you check the element token again? The issue seems to be the lowercase P in the @p2#

StevenVanDyk commented 3 years ago

$QD8VR8o~@P2#

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:31 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

Ok, it's working now. Can you check the element token again? The issue seems to be the lowercase P in the @p2https://github.com/p2#

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828683408, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDF2ZXLIDZUJCCWX5RTTLBH7HANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

if you revoke and request a new one, does it have a capital @P2# at the end or lowercase @p2#

zarmstrong commented 3 years ago

Oh it looks like github is making it lowercase. Damnit github! gotta put the token inside of ` marks to prevent it from lowercasing it

StevenVanDyk commented 3 years ago

Yeah they all seem to

$sDBy520~@P2# Is the latest

I tried revoking and creating a couple times

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:35 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

if you revoke and request a new one, does it have a capital @P2# at the end or lowercase @p2#

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828686149, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDFBKBLXRQFLAM5DS3TTLBIM7ANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

Now that that's out of the way, let me dig in some more with that token lol

StevenVanDyk commented 3 years ago

$sDBy520~@P2# @.***D73C3B.F9FD67F0] As an image as well so it cant “help us” with the text

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:36 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

Oh it looks like github is making it lowercase. Damnit github! gotta put the token inside of ` marks to prevent it from lowercasing it

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828687010, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDA3X5FPHG4UF7RQNOLTLBIRJANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

I was able to import the PC with the token $sDBy520~@P2#

StevenVanDyk commented 3 years ago

Ok I get the Conversion log @.D73C3D.07BC94F0]The Character name updates on the actor list But the Sheet does not And if I close the sheet I cannot reopen it .. @.D73C3D.07BC94F0] @.***D73C3D.07BC94F0]

I have all other modules off and I’m using the 0.7.9 release.

Any thoughts on where I could look?

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:41 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

I was able to import the PC with the token $sDBy520~@P2#

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828690523, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDBEGKNIC4KTD3FQMKLTLBJDVANCNFSM43XOI2CQ.

StevenVanDyk commented 3 years ago

😐

I have been checking for updates on the modules And the base version I missed checking for updates o the pf2nd :/

I just went back to look through it and Did the update and it now imported.

I’ll add that on my update list 😐

Thanks for the help its much appreceated

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:41 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

I was able to import the PC with the token $sDBy520~@P2#

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828690523, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDBEGKNIC4KTD3FQMKLTLBJDVANCNFSM43XOI2CQ.

StevenVanDyk commented 3 years ago

One odd item

When importing his ac is really impressive @.***D73C4A.50D67F40]

In hlo is @.***D73C4A.50D67F40]

When I goto the inventory for him and unequit his leather armor it drops to 13 Equipping the armor again brings it back to 1032

It is imported over (I tried with a test character so empty base) For now I just added it back in with in foundry.

Thanks again

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 2:41 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

I was able to import the PC with the token $sDBy520~@P2#

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828690523, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDBEGKNIC4KTD3FQMKLTLBJDVANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

The pictures don't come through from email responses to this. Can you add them directly in GitHub?

StevenVanDyk commented 3 years ago

Will do

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: @.> Sent: Wednesday, April 28, 2021 6:58 PM To: @.> Cc: @.>; @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

The pictures don't come through from email responses to this. Can you add them directly in GitHub?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828833979, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDAPIZAF52IDALHJ6SLTLCHIBANCNFSM43XOI2CQ.

StevenVanDyk commented 3 years ago

One odd item hlo ac ac

When importing his ac is really impressive

In hlo is

When I goto the inventory for him and unequit his leather armor it drops to 13 Equipping the armor again brings it back to 1032

It is imported over (I tried with a test character so empty base) For now I just added it back in with in foundry.

Thanks again

zarmstrong commented 3 years ago

Pushed out some changes that should fix this. You will need to update the module. Let me know how it looks after you update.

StevenVanDyk commented 3 years ago

Just got a Chance to test it out And works perfect

Thanks 😊

From: zarmstrong @.> Sent: Wednesday, April 28, 2021 10:46 PM To: zarmstrong/hlo-importer @.> Cc: StevenVanDyk @.>; Author @.> Subject: Re: [zarmstrong/hlo-importer] Unable to import character (#7)

Pushed out some changes that should fix this. You will need to update the module. Let me know how it looks after you update.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/zarmstrong/hlo-importer/issues/7#issuecomment-828905128, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APGGYDGJOSESUROCJB5RVPLTLDB5XANCNFSM43XOI2CQ.

zarmstrong commented 3 years ago

Great! Closing this