gd-utils icon indicating copy to clipboard operation
gd-utils copied to clipboard

fix some problem

Open liaojack8 opened this issue 4 years ago • 11 comments

  1. 新增了功能讓用戶可以自定義按鈕列數, 即每列一個或每列兩個
  2. 原先在tg.js中有一處誤植, 已修正 這邊已經改用簡體版本 btw. 我是tg上的@jkliao, 謝謝大神開發這套工具: )

liaojack8 avatar Jul 06 '20 18:07 liaojack8

😱71个commit... 能不能搞成干净点的pr啊,能一眼看到哪些文件变化了的。。不然我没法review……

iwestlin avatar Jul 07 '20 04:07 iwestlin

modify with your latest version at master, pls check it. Let users can set the num of button per cols, and the path of SA json files. Fix the mistake in ./src/tg.js 'auth.js' at line 12.

liaojack8 avatar Jul 07 '20 11:07 liaojack8

config.js这个文件是写在.gitignore里的,而且用户自己也做了修改,如果再把它的修改提交进git,那么下次用户更新代码的时候可能会引发合并冲突……

iwestlin avatar Jul 07 '20 11:07 iwestlin

还有我没在代码变动里看到设置sa文件路径的地方啊。。

iwestlin avatar Jul 07 '20 11:07 iwestlin

修正了sa路徑設置項 至於合併衝突的話, 可能需要讓用戶手動配置 或是請您再考慮這個設置項的存留了

liaojack8 avatar Jul 07 '20 12:07 liaojack8

SA_PATH 只是导出了,我没看到在代码里用到它?

iwestlin avatar Jul 07 '20 12:07 iwestlin

已經更新.

liaojack8 avatar Jul 07 '20 12:07 liaojack8

我考虑了下,以防用户后续更新产生冲突,这个PR先不合并了啊……不好意思啊让你白白辛苦🙏

然后其实可以不用修改config的,sa_path可以放进命令行参数里;button列数这个也可以放到数据库里,比如新建一个config的表,让用户可以直接在机器人里设置。

如果你有兴趣修改,建议从最新代码fork出去,这样不至于产生70多个commit……

iwestlin avatar Jul 07 '20 13:07 iwestlin

不會的 全部寫在config是讓大家有個統一的修改入口 沒考慮到會引發衝突 至於這70多個是我fork過來做繁化的過程 想發pr但已經沒辦法再fork一份 只好開了一個branch再fetch pull你的新版本 改完再送pr 真是折騰死我了 之後有問題我還是發issue吧

liaojack8 avatar Jul 07 '20 13:07 liaojack8

这个sa路径放到config里真的很有必要! 已经安装的用户提示重新配置或者不需要跟新 我也可以搞个脚本把他的config更新一遍

vitaminx avatar Jul 17 '20 08:07 vitaminx

用命令参数只是权宜之计 这样虽然看似解决了问题 但是使用期起来极为不方便(每次都要输那么一长串路径)

vitaminx avatar Jul 17 '20 08:07 vitaminx