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

Include the required wkof.Menu module #8

Merged
merged 1 commit into from
Feb 11, 2025
Merged

Conversation

SimonSapin
Copy link
Contributor

Jitai 3.3.5 failed to load for me: WaniKani is as if Jitai was not installed at all, and the browser console has an error:

Uncaught (in promise) TypeError: Unhandled Promise Rejection: wkof.Menu is undefined

This change fixes it for me. It was a guess based on Menu only being mentioned in supported_modules in wkof’s code, and https://community.wanikani.com/t/wanikani-open-framework-developer-thread/22231/2 showing wkof.include(…)

Jitai 3.3.5 failed to load for me: WaniKani is as if Jitai was not installed at all, and the browser console has an error:

```
Uncaught (in promise) TypeError: Unhandled Promise Rejection: wkof.Menu is undefined
```

This change fixes it for me. It was a guess based on `Menu` only being mentioned in `supported_modules` in wkof’s code, and https://community.wanikani.com/t/wanikani-open-framework-developer-thread/22231/2 showing `wkof.include(…)`
@marciska
Copy link
Owner

Interesting! For me it even works without. Sounds to me like an incompatibility with wkof, maybe you are using an old version? My latest version of WKOF installed is v1.2.10.
Anyway, I confirmed it working with and without, so I'll just include it 🙂

@marciska marciska merged commit 79b3b32 into marciska:main Feb 11, 2025
1 check passed
@SimonSapin SimonSapin deleted the patch-1 branch February 11, 2025 05:45
@SimonSapin
Copy link
Contributor Author

The error I see is also with wkof 1.2.10.

If multiple scripts depend on wkof, do they share one global wkof object? Is it possible you have another script that includes the Menu module?

Either way, thanks for the new release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants