New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JSON import mix up for Batman collab and Shinrabansho #741

Closed
grosdodo opened this Issue Sep 9, 2015 · 7 comments

Comments

Projects
None yet
3 participants
@grosdodo
Copy link

grosdodo commented Sep 9, 2015

Hi again, I've seen a couple of issues already dealing with the Batman/Shinra mix up.
However, JSON import still has an issue : Shinra cards are imported as Batman cards
and vice versa. Any chance you'll fix this ? Thanks !

@Tito151

This comment has been minimized.

Copy link

Tito151 commented Oct 4, 2015

In my PADHerder, King Saiga is showing up as Catwoman, but it's just the picture. The PADX links and ATT sorting still works properly.

@madcowfred

This comment has been minimized.

Copy link
Owner

madcowfred commented Oct 11, 2015

Hm, I just went in to the code to fix the JSON import thing and it looks like it should work... for NA people. Is your account set to something else?

@grosdodo

This comment has been minimized.

Copy link

grosdodo commented Oct 11, 2015

Yeah, I'm in EU, want me to give it a try ?

@grosdodo

This comment has been minimized.

Copy link

grosdodo commented Oct 11, 2015

I can confirm it still doesn't work for EU.
The import log says #671 Sacred Dragon King Saiga has been created.
The monster box shows it as #1049 US. Sacred Dragon King Saiga and links to
padx #1049.
(#671 being Catwoman)

@madcowfred madcowfred closed this Oct 11, 2015

@madcowfred

This comment has been minimized.

Copy link
Owner

madcowfred commented Oct 11, 2015

Should be fixed now.

@grosdodo

This comment has been minimized.

Copy link

grosdodo commented Oct 11, 2015

Indeed, thanks :)

@madcowfred madcowfred added the bug label Oct 11, 2015

@Tito151

This comment has been minimized.

Copy link

Tito151 commented Oct 14, 2015

All good for me too! Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment