Skip to content
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

Deck containing card not yet released in English fails to export to YGOPRODECK #18

Open
kevinlul opened this issue Sep 2, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@kevinlul kevinlul added the bug Something isn't working label Sep 2, 2024
@kevinlul
Copy link
Contributor Author

kevinlul commented Sep 2, 2024

image

Bad URL: ydke://4GakAOBmpAARtIkBEbSJARG0iQHkuA4D5LgOA+S4DgO3HUMFTGhJAExoSQCF1okFhdaJBYXWiQUo9ZkBsCPeALAj3gCwI94AKpVlASqVZQEqlWUBqfKzBaCUBAIDwk4EqKXeBail3gWopd4Fx6sgA8erIAOYGscAmBrHAP7WVAJqSB4AakgeAGpIHgAAAAAAAAAAAAAAAAAAAAAAHjeCAeqY0QRr/yQFIkiZACJImQAiSJkAUr0tAw==!iptnBEiIswBr1MwEa9TMBDW0vgQ1tL4E0fqMBQAAAAAAAAAA66qLBc3gIQCNJ5gD3PhoAdcV3wUAAAAA!/omcBf6JnAX+iZwF1xqfAdcanwEAAAAAAAAAAAAAAACEJX4AhCV+AIQlfgD73BQBXWneAl1p3gJ+iUMD!

@kevinlul
Copy link
Contributor Author

kevinlul commented Sep 2, 2024

# 粛声アザミナドラグマ
# https://www.db.yugioh-card.com/yugiohdb/member_deck.action?ope=1&wname=MemberDeck&ytkn=6b30f387b69c2524e84ae5f0a9e5366fb113aa88ba5d39de044d846f926f907b&cgid=c42baa47b8670a7d3ae426989a9b4843&dno=128&request_locale=en
#main
10774240
10774240
25801745
25801745
25801745
51296484
51296484
51296484
88284599
4810828
4810828
92919429
92919429
92919429
26866984
14558128
14558128
14558128
23434538
23434538
23434538
95679145
33854624
72270339
98477480
98477480
98477480
52472775
52472775
13048472
13048472
39114494
1984618
1984618
1984618




25311006
80845034
86310763
10045474
10045474
10045474
53329234
#extra
73898890
11765832
80532587
80532587
79606837
79606837
93125329


93039339
2220237
60303245
23656668
98506199

!side
94145022
94145022
94145022
27204311
27204311



8267140
8267140
8267140
18144507
48130397
48130397
54757758

blank lines present

@kevinlul
Copy link
Contributor Author

kevinlul commented Sep 2, 2024

Likely a data issue with the Konami IDs not being present in the API

@kevinlul
Copy link
Contributor Author

kevinlul commented Nov 2, 2024

Mitigation: filter out undefined from the arrays here

const deckPasswords = {
main: deckKonamiIds.main.map(kid => cards.get(kid)),
extra: deckKonamiIds.extra.map(kid => cards.get(kid)),
side: deckKonamiIds.side.map(kid => cards.get(kid))
};
console.log(deckPasswords);

Proper fix: improve data quality of API

@kevinlul
Copy link
Contributor Author

kevinlul commented Nov 2, 2024

Tests still fail because they're not expecting to read an export with under 40 cards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant