search

kelciour / improved-quizlet-to-anki-21-importer

21 stars 7 forks source link

Any plan to support importing a class? #2

Open liutiming opened 3 years ago

liutiming commented 3 years ago

really a nice add on for importing set. how about class? The add-on seems to confuse between classes and sets as it downloads the set xxxx instead of class xxxx when xxxx is in the url.

kelciour commented 3 years ago

Thank you, but unlikely it'll be me. I added support for Quizlet folders about a month ago, it seems to work, but not very interested to spend more time on the add-on anytime soon.

liutiming commented 3 years ago

adding further details for people who want to add support for directly importing class: it seems that requests.get's output from quizlet folder contains the set url that can be directly used for downloading. Such info does not exist in the requests.get output from quizlet class though...

@kelciour if you remember the emails communications we had at the start of the year, I think we can add functionality from there to support downloading of multiple url and add a parent deck name? The urls of the sets from a class can be easily extracted from a browser add-on (I am using Linkgopher in Firefox) so that is probably the most convenient (though less elegant) way that I can think of to import all sets in a class.

kelciour commented 3 years ago

Hi Timing. Yes, I remember now. At the moment, as long as the add-on is not broken, the add-on unlikely will be updated by me, but if someone sends a pull request with the changes, I'll be happy to accept it pretty much as it is, update the add-on on AnkiWeb and mention the pull request. In the past, if it were some major changes or frequent updates, I'd likely decline it and instead would suggest to upload a new version on AnkiWeb so I could mention it in the add-on's description. But now the add-on is on GitHub, there probably will be no much difference for me, and I'll be happy with any option.