-
Notifications
You must be signed in to change notification settings - Fork 94
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
improve dictionary validation #520
improve dictionary validation #520
Conversation
✔️ No visual differences introduced by this PR. View Playwright Report (note: open the "playwright-report" artifact) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but merge conflict
@Scrub1492 @stephenmk I noticed that the latest version of Jitendex https://github.com/stephenmk/Jitendex/releases/tag/v4.1-1 is now failing to validate. Do we know why?
|
I can't reproduce that error. I just built the extension locally using the code from commit 39265a4 and installed it in firefox and chromium. Neither one had a problem verifying and importing the jitendex zip file from that release page. |
You're right, sorry for the false alarm -- I had an old schema locally and had to ./build.sh again. |
Solves #507 (dev dictionary validation now validates files with skipped indexes).
Also changed the
QueryDetails
in dictionary-import.js_getArchiveFiles
to be a simple 2d array as opposed to a Map because it is more performant when looped over.