Closed waynecz closed 6 years ago
更新之后每次打开都弹出更新内容的提示 , 这个可以关闭吗?
@lyhzh 我明天修复更新下🤣
不好意思 更新内容提示 我没看到这里有提到 给单开了一个 issue。。
@waynecz
chrome-app-samples/samples/storage/main.js:40
/**
* @summary 插件第一次安装时设置初始值
*/
chrome.runtime.onInstalled.addListener(async reason => {
if (reason.reason !== 'update') {
Storage.set(TR_SETTING_BLACK_LIST_KEY, {})
Storage.set(TR_SETTING_HAS_TOAST_KEY, true)
Storage.set(TR_SETTING_IS_DIRECTLY_KEY, false)
Storage.set(TR_SETTING_SKIP_CHINESE_KEY, true)
Storage.set(TR_SETTING_SHANBAY, false)
Storage.set(TR_SETTING_AUTO_SPEAK, false)
Storage.set(TR_SETTING_ENGLISH_MEANING, true)
Storage.set(TR_SETTING_KEYBOARD_CONTROL, false)
Storage.set(TR_SETTING_FONT_FAMILY, 'song')
Storage.set(NOTICE_IS_RECEIVED, false)
}
})
chrome.notifications.clear('updateInfo')
/**
* @summary 检查和接受通知
*/
if (Storage.get(NOTICE_IS_RECEIVED)) {
chrome.notifications.create('updateInfo', {
iconUrl: 'http://p5grwrmf4.bkt.clouddn.com/dadda-ico.png',
type: 'basic',
title: 'v1.1.0 更新',
message: '扇贝单词同步 / 快捷键控制 / 简单的翻译框... 点击查看更多更新内容',
priority: 2,
eventTime: Date.now() + 100000
})
Storage.set(NOTICE_IS_RECEIVED, true)
}
这样写可以运行吗? 另外我觉得如果可以的话,可以通过网络请求来获得具体的通知内容,不应该直接写在这里
@lyhzh 这样有个问题是已安装的用户不会执行 Storage.set(NOTICE_IS_RECEIVED, false)
, 因为在首次安装的执行条件里,我打算给写到更新事件里
if (reason.reason === 'update') {
// ... notification
}
嗯嗯,更新内容放到网络上配置可行,我弄一下
@waynecz 嗯 , 大佬加油 👍
@j0ck1e @bigbigbo @chinafuhai @neilsustc @Jface001
更新具体内容