naive-ui-admin icon indicating copy to clipboard operation
naive-ui-admin copied to clipboard

bug记录

Open Mr-BeanSir opened this issue 1 year ago • 15 comments

此issues记录碰到的bug,后续会逐一修复pr

Mr-BeanSir avatar Oct 19 '24 16:10 Mr-BeanSir

CreateDrawer 没有暴露openDrawer函数

添加子菜单时似乎没有传递便捷的有关父菜单的key,没法得知子菜单添加给谁

Mr-BeanSir avatar Oct 19 '24 16:10 Mr-BeanSir

依赖似乎少了lodash,多了lodash-es

考虑统一修改为lodash-es,需要修改loadChunk.ts文件

Mr-BeanSir avatar Oct 19 '24 16:10 Mr-BeanSir

src/views/system/menu/menu.vue中onMounted的多余代码

    // const keys = treeMenuList.result.sys_menu.map((item) => item.key);
    // Object.assign(formParams, keys);

同时 const treeMenuList = await getSystemMenu(); 需要二次封装以便添加菜单后子路由控制重载list

文字错误:handleDel函数 您确定想删除此权限吗 应修改为 您确定想删除此菜单吗 修改变量名易于理解:function selectedTree(keys: []) 修改为 function selectedTree(currentSelectKey: [])

Mr-BeanSir avatar Oct 20 '24 14:10 Mr-BeanSir

selectedTree 没有 delete formParams['children']; 这样选择父目录重置时候会导致children保留

Mr-BeanSir avatar Oct 20 '24 16:10 Mr-BeanSir

src/layout/index.vue 返回顶部在移动端会遮挡元素 修改为 <n-back-top :right="10" :bottom="10" /> 可选

const currentRoute = useRoute(); 提取出来,而不是放在computed

src/store/modules/asyncRoute.tsfilter.map((node: any) => ({ ...node })) 似乎没有意义,希望作者看到能够回复告知一下作用

Mr-BeanSir avatar Oct 21 '24 12:10 Mr-BeanSir

src/components/Table/src/hooks/useDataSource.ts 多余的 const results = res[listField] ? res[listField] : [];

用户权限管理设置权限存在严重bug,tree默认读取key,但是key单纯是key,权限已经变成了permission,就是说permission和key必须相同

Mr-BeanSir avatar Oct 21 '24 16:10 Mr-BeanSir

src/utils/index.ts 函数修改为,treeAll在外面会导致多次全选权限重复

export function getTreeAll(data: any[]): any[] {
  const treeAll: any[] = [];
  data.map((item) => {
    treeAll.push(item.key);
    if (item.children && item.children.length) {
      treeAll.push(...getTreeAll(item.children));
    }
  });
  return treeAll;
}

Mr-BeanSir avatar Oct 23 '24 12:10 Mr-BeanSir

failed navigation Error: Avoided redundant navigation to current location: "/login?redirect=/welcome".

出现场景:

whitePathList = ['/', 'login']

首页 / 跳转到 login,提交表单后 replace 到 welcome 页面,不出现。 在 login 页面 重新刷新页面,提交表单后出现以上错误。

cleveng avatar Oct 24 '24 09:10 cleveng

        <template #action>
          <TableAction />
        </template>

似乎多余,待求证

Mr-BeanSir avatar Oct 24 '24 15:10 Mr-BeanSir

上面提到也遇到好几个都算比较好改的,提一个比较重要的,就是菜单的数据结构和路由数据结构完全不同,改成后端动态路由权限,要修改的东西比较多。希望作者优化优化,不然对新手很不友好,毕竟用这种模板的后端偏多,自己改动太多太浪费时间了。

ninjashixuan avatar Oct 30 '24 09:10 ninjashixuan

上面提到也遇到好几个都算比较好改的,提一个比较重要的,就是菜单的数据结构和路由数据结构完全不同,改成后端动态路由权限,要修改的东西比较多。希望作者优化优化,不然对新手很不友好,毕竟用这种模板的后端偏多,自己改动太多太浪费时间了。

菜单和路由数据结构,本身是相似,但是不完全相同,另外动态路由压根不需要修改源码,前提是需要按照约定的 mock 数据格式返回(这个格式是给动态创建路由使用的,和菜单没有关联) 建议参考一下文档哈

jekip avatar Oct 30 '24 10:10 jekip

上面提到也遇到好几个都算比较好改的,提一个比较重要的,就是菜单的数据结构和路由数据结构完全不同,改成后端动态路由权限,要修改的东西比较多。希望作者优化优化,不然对新手很不友好,毕竟用这种模板的后端偏多,自己改动太多太浪费时间了。

菜单和路由数据结构,本身是相似,但是不完全相同,另外动态路由压根不需要修改源码,前提是需要按照约定的 mock 数据格式返回(这个格式是给动态创建路由使用的,和菜单没有关联) 建议参考一下文档哈

可能理解不同,我之前用的,菜单就是整个动态要返回的路由,权限控制这块就是控制每个人显示不同的菜单,所以觉得mock的动态路由数据结构和菜单表单的相差有点大觉得奇怪。

ninjashixuan avatar Oct 31 '24 01:10 ninjashixuan

上面提到也遇到好几个都算比较好改的,提一个比较重要的,就是菜单的数据结构和路由数据结构完全不同,改成后端动态路由权限,要修改的东西比较多。希望作者优化优化,不然对新手很不友好,毕竟用这种模板的后端偏多,自己改动太多太浪费时间了。

菜单和路由数据结构,本身是相似,但是不完全相同,另外动态路由压根不需要修改源码,前提是需要按照约定的 mock 数据格式返回(这个格式是给动态创建路由使用的,和菜单没有关联) 建议参考一下文档哈

v2开源版是否有文档?目前仅查到v1文档,但内容过于落后,似乎v2仅收费版有文档

Mr-BeanSir avatar Oct 31 '24 16:10 Mr-BeanSir

打包后访问,跳转路由后会出现白屏 #300

YepYuYu avatar Jan 12 '25 14:01 YepYuYu

n-modal 下的 useECharts 无法显示,正常页面的 onMount 可以渲染组件,但是在n-modal弹窗下的图表无法显示。

cleveng avatar Jan 26 '25 12:01 cleveng