Duplicates about mdanki

[ewnd9]

Hey all, sorry for the long time. I've released [email protected] with the patch from @amorriscode 👍

[ashlinchak]

Thanks, @amorriscode, and @DHNishi for the investigation and the PR. I'll update the dependency when the PR'll be merged to anki-apkg-export

[amorriscode]

@ashlinchak I made a PR. 😄

[DHNishi]

This appears to be an issue in the underlying library. repeat-space/anki-apkg-export#37 calls attention to it. Essentially, https://github.com/repeat-space/anki-apkg-export/blob/master/src/exporter.js#L99 appears to show it using the timestamp of generation to create the card ID. I think if it switched to using something like a hash of the deckName and the card front, you could probably avoid the duplicate problem.

[amorriscode]

@DHNishi thanks for the investigation. I made a PR over there. We'll see if it gets in!

[amorriscode]

So I realized now that the update of anki-apkg-export doesn't quite solve the problem. The issue being that existing decks don't get loaded and updated. They are just completely regenerated.

[ashlinchak]

Yeah, just checked it :) I believe having this is not enough as those variables are generated based on the topDeckId which is generated based on Date.now() anyway.

[amorriscode]

🤦‍♂ Sorry about that. I suppose the topDeckId should be based off of the deck name itself?

[amorriscode]

Made a new PR based on some of our ideas. Tested it with mdanki this time.

Not sure if it handles nested decks; that will require more research but it's a start.