NewEconoLab / nel-wallet-vue

NEL Web Wallet based on vue Framework
7 stars 3 forks source link

down下来跑,报错 #1

Open treasureDouDou opened 6 years ago

treasureDouDou commented 6 years ago

image

vincentOpenSource commented 6 years ago

你有执行npm install 吗,会自动帮你把依赖安装好

treasureDouDou commented 6 years ago

当然执行了,这句错误的地方我给你注释掉了,现在跑起来,看看效果和代码

vincentOpenSource commented 6 years ago

这是转字节的地方,统一处理合约返回的数据,你注了就用不了 自己下载一下 buffer的依赖

vincentOpenSource commented 6 years ago

我一会儿更新这边的方法用原有sdk里有的方法处理

treasureDouDou commented 6 years ago

@vincentOpenSource 依赖插件名是什么啊,这个没有放到package吗,那每个人没安装都要来问一次啦

vincentOpenSource commented 6 years ago

放到package中了

vincentOpenSource commented 6 years ago

不清楚为何你没下下来

vincentOpenSource commented 6 years ago

我已经更换方法了,一会儿测试后推上了,你也不需要下载Buffer

treasureDouDou commented 6 years ago

几个疑问: 1.不是有router-link吗?为何要自己写一个VLink 2.路由不是有vue-router吗,为什么要在main.ts用计算属性去返回对应的组件来显示相应的页面。计算属性也不是这么用的啊,也不能按需加载,万一写移动端适配了,第一次不是加载几M,流量遭不住 3.写class的语法是为了什么,还不如直接写react+ts, ts更支持react吧,你写模板的时候ts也搞不了事情啊 4.确实上手比较困难,看不清层次

vincentOpenSource commented 6 years ago

方法已经修改过并推上git了。关于以上四个问题 1.由于刚上手前端项目对vue的各个组件并不是特别熟悉。只是写了个简单的路由。 2.对于路由会进行进一步的优化,技术上的问题可能还需要请教一下,是否改用成require就可以实现按需加载 3.ts的语法更据有结构化,对Neo支持的SDK也都是ts的,确实考虑下个版本会使用react+ts 4.本人的架构经验缺乏,目前层次主要就是 src 下 components 是公用的组件 layouts 是页头和页面主体的嵌套。page 是每个页面的单独业务 tools则是各个使用到的封装的工具类,tools下的entity.ts存放实体类。 谢谢您的提议

treasureDouDou commented 6 years ago

懒加载老版本写法 image 新版本写法 https://router.vuejs.org/zh/guide/advanced/lazy-loading.html

vincentOpenSource commented 6 years ago

好的谢谢

QingmingZi commented 6 years ago

感谢你的真知灼见^_^ @treasureDouDou

QingmingZi commented 6 years ago

不同问题和建议(特别是不同类型的),建议分不同issue @treasureDouDou

treasureDouDou commented 6 years ago

@QingmingZi 好的

QingmingZi commented 6 years ago

也就是一看标题,能够知道大致内容,方便其他社区成员浏览