Skip to content

fix: uninstall default and import problem#267

Merged
scarqin merged 6 commits into
mainfrom
yZaio
Mar 14, 2023
Merged

fix: uninstall default and import problem#267
scarqin merged 6 commits into
mainfrom
yZaio

Conversation

@yZaio

@yZaio yZaio commented Mar 13, 2023

Copy link
Copy Markdown
Contributor

fix: uninstall default and import problem

@scarqin scarqin changed the title Yzaio fix: uninstall default and import problem Mar 13, 2023
import { WebExtensionService } from './webExtension.service';

const defaultExtensions = ['postcat-export-openapi', 'postcat-import-openapi'];
const uninstallDefaultExtensions = StorageUtil.get('uninstall_default_ext_list');

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里可以优化成第一次打开才安装默认插件,不需要缓存卸载信息。
image

mixin: (data, base) => {
const handleData = {};
for (let item of data) {
handleData[item.name] = {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里没有考虑子参数逻辑(childList)
image

table 的遍历逻辑可以看看 filterTableData 能否被复用。

@yZaio yZaio closed this Mar 14, 2023
@scarqin scarqin reopened this Mar 14, 2023
@scarqin scarqin merged commit 6cbe019 into main Mar 14, 2023
@scarqin scarqin deleted the yZaio branch March 14, 2023 07:20
yZaio added a commit that referenced this pull request May 5, 2023
* fix: uninstall default and import
---------

Co-authored-by: sunzhouyang <sunzhouyang@eolink.com>
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