-
Notifications
You must be signed in to change notification settings - Fork 223
Move util database modification functions #499
Move util database modification functions #499
Conversation
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.
The descriptive error messages for stuff like importing a duplicate dictionary don't work:
Yomichan v20.4.18.0 has encountered a problem.
Originating URL: chrome-extension://jpjldhmgaijbpomlojkhkghbbebbiblb/bg/settings.html
Error
Issues can be reported at https://github.com/FooSoft/yomichan/issues
@@ -809,6 +811,11 @@ class Backend { | |||
return portName; | |||
} | |||
|
|||
async _onApiImportDictionaryArchive({archiveContent, details}, sender, onProgress) { | |||
this._validatePrivilegedMessageSender(sender); |
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.
Should this be moved to the handler info object like in #497 (separately)?
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.
Probably, now that there's somewhat of a convention for it.
Error serialization problems caught me again. c062ef1 |
apiImportDictionaryArchive
andapiDeleteDictionary
replaceutilDatabaseImport
andutilDatabaseDeleteDictionary
.Related: #455