pavelgordon / babbel2anki-chrome-extension

Chrome Extension which imports and syncs Babbel vocabulary to Anki. Supports auto-syncing, customising deck and card template for syncing.
https://chrome.google.com/webstore/detail/babbel2anki/dnnognkhllcinkeimdhdchjggploankm?hl=en
MIT License
18 stars 1 forks source link

Sync with Anki button does not appear #17

Open tonyhogsten opened 2 years ago

tonyhogsten commented 2 years ago

Hello!

Is this project still maintained? I've followed the instructions as you've lined them out, but to me, the Sync with Anki button does not show up at all.

Anki versions: Version ⁨2.1.52 (ab1c2395)⁩ Python 3.9.7 Qt 5.15.2 PyQt 5.15.5

Chrome version: Version 102.0.5005.115 (Official Build) (64-bit)

Plugin version: 1.0.2

I've verified that http://[localhost:8765](http://localhost:8765/)/ gives me the response: AnkiConnect v.6

I have configured the AnkiConnect plugin with

{
    "apiKey": null,
    "apiLogPath": null,
    "ignoreOriginList": [],
    "webBindAddress": "127.0.0.1",
    "webBindPort": 8765,
    "webCorsOrigin": "http://localhost",
    "webCorsOriginList": [
        "*"
    ]
}

I'm not getting any clear errors in the console:

image

And for me Babbel looks like this:

image

Is there any way you could help me out?

Thanks

timpaulus commented 1 year ago

Hello!

I can confirm that I had the same problem even after thoroughly following the installation steps in the README.

I decided to look into it and it seems that Babbel slightly changed the layout of the review manager page, including the html class names currently used to verify that the required html elements are loaded in the DOM before injecting the sync button. At least this is my guess, since I don't know how the html looked before.

I slightly changed how the contentscript handles the new html class names and toolbar layout. The sync button now appears for me.

I'm not sure if this is welcomed, but I created a pull request for the fix. @pavelgordon Could you take a look at it and confirm the fix?

All the best

Jannik098 commented 1 year ago

That would be awesome! I'm having the same issue!

pavelgordon commented 1 year ago

Hello! Thanks a lot for the contribution. Took a look and fix works, so i merged it, and will upload it to be available to be installed via Google Chrome Extensions once they approve it.

wuqui commented 7 months ago

Unfortunately, the sync button is gone for me now. It still worked fine yesterday. Maybe this is due to changes in Chrome? It would be great to have a fix, I use this extension every day.

Btw., thanks for making this extension 🙂

timpaulus commented 7 months ago

@pavelgordon The layout changed again and the html class names are now minimized/obfuscated by Babbel. I created a PR which fixes the issue by using an attribute selector. I don't know how long this fix will work, but at least it's something. Could you take a look and maybe merge it again?

@wuqui If you need the fix urgently you can just download the master.zip from my fork and install the Extension manually :)

wuqui commented 7 months ago

@wuqui If you need the fix urgently you can just download the master.zip from my fork and install the Extension manually :)

Thanks! Where can I find it?

wuqui commented 7 months ago

@timpaulus: I guess you meant app.zip instead of master.zip, right? I downloaded and installed it now, but the sync button still doesn’t appear. Maybe because Chrome shows an error for the plugin?

image
epiwatson7 commented 6 months ago

@pavelgordon, sync to anki button is gone again. I think the layout of the Review page changed on Babel. can you tweak your extension pretty please?

wuqui commented 5 months ago

The above solution using the forked version by @timpaulus had worked for me until today. As of today, the sync button is gone again, unfortunately. Seems like Babbel changed it’s design and that made it disappear again. Could someone take care of this again or let me know how I can fix it myself, please? :)

timpaulus commented 5 months ago

Oof, shortly after my fix was merged into master... 😒 I quickly implemented another fix. Only a slight change was necessary.

@wuqui You can again install the extension by using my master branch: https://github.com/timpaulus/babbel2anki-chrome-extension After downloading the files, it is important that you zip the app folder yourself and add it manually as an extension. Do not use the existing app.zip!

@pavelgordon This is the third time a fix for the missing sync button is needed. I propose that the sync button is moved into the popup window of the extension. At least additionally to the existing solution. This way it's more stable against page layout changes from Babbel. What do you think? If you want I can implement it and create a PR.

Also, since Manifest V2 is deprecated by Google in June 2024, an update to Manifest V3 is soon necessary. https://developer.chrome.com/docs/extensions/develop/migrate/mv2-deprecation-timeline I don't know what your plans for this extension are and if you want to do an overhaul yourself. If you don't find the time I'd like to offer my help and migrate this extension. But since this will require major changes I first want to ask for permission to do so?

wuqui commented 5 months ago

@wuqui You can again install the extension by using my master branch: https://github.com/timpaulus/babbel2anki-chrome-extension After downloading the files, it is important that you zip the app folder yourself and add it manually as an extension. Do not use the existing app.zip!

Thanks so much, it works! 🙂

No need to zip the app folder, btw., one can simply do “load unpacked” in the extensions manager in Chrome.

pavelgordon commented 5 months ago

I will merge it today!

On Sat, 3 Feb 2024 at 12:14, wuqui @.***> wrote:

@wuqui https://github.com/wuqui You can again install the extension by using my master branch: https://github.com/timpaulus/babbel2anki-chrome-extension After downloading the files, it is important that you zip the app folder yourself and add it manually as an extension. Do not use the existing app.zip!

Thanks so much, it works! 🙂

No need to zip the app folder, btw., one can simply do “load unpacked” in the extensions manager in Chrome.

— Reply to this email directly, view it on GitHub https://github.com/pavelgordon/babbel2anki-chrome-extension/issues/17#issuecomment-1925283850, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASZF3AQDBG74BUSRX6RNGDYRYL2TAVCNFSM5ZNGE4BKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJSGUZDQMZYGUYA . You are receiving this because you were mentioned.Message ID: @.***>

wuqui commented 2 weeks ago

The button is gone again, unfortunately 🫤

J4K0B commented 1 week ago

The button is gone again, unfortunately 🫤

I created a pull request that fixes the issue :)

pavelgordon commented 1 week ago

The button is gone again, unfortunately 🫤

I created a pull request that fixes the issue :)

Merged, thanks! I will upload it to chrome as well, will take a day-two to approve them

wuqui commented 1 week ago

Great, thank you both!