Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix some problem #74

Open
wants to merge 74 commits into
base: master
Choose a base branch
from
Open

fix some problem #74

wants to merge 74 commits into from

Conversation

liaojack8
Copy link

@liaojack8 liaojack8 commented Jul 6, 2020 •
edited
Loading

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

vitaminx added 30 commits June 30, 2020 21:20
@vitaminx
修改说明
5f323ce
@vitaminx
跟新更高效的脚本
2ebdf2d
@vitaminx
一键脚本改版
322417a 
脚本改版说明：
1-对脚本进行改版，效率更高代码更简洁
2-修复了有些系统不能识别的BUG
2-增加了输入“机器人token/telegram账号名/WEB服务名/网址”信息对错的判断

一件脚本使用方法：
之际复制以下链接到VPS命令行窗口粘贴回车即可以执行
@vitaminx
Merge pull request vitaminx#5 from vitaminx/vitaminx-patch-gdutilsins…
6bd7c5c 
…tall.sh-2.0

一键脚本改版
@vitaminx
Delete gdutilsinstall.sh
a4daca2
@vitaminx
改版脚本错误修复
dec3202 
✅脚本改版说明：
1-对脚本重新编写，效率跟高代码更少
2-修复了有些系统不能识别的BUG
3-增加了输入“机器人token/telegram账号名/WEB服务名/网址”信息对错的判断
4-修复了两处小错误
@vitaminx
Merge pull request vitaminx#6 from vitaminx/vitaminx-patch-gdutilsins…
7919b11 
…tall.sh-2.1

改版脚本错误修复
@vitaminx
nginx配置格式修改
9026cab
@vitaminx
Update gdutilsinstall.sh
a86fdb3
@vitaminx
Update gdutilsinstall.sh
0c86a64
@vitaminx
分脚本
a1264df 
针对有些人只要查询转存和机器人两部分功能中的一部分，把脚本分开了两个一键部署
gdutilscsinstall.sh为查询转存一键部署脚本
gdutilsbotinstall.sh为机器人一键部署脚本
@vitaminx
增加分脚本说明
4eb211b
@vitaminx
Update gdutilscsinstall.sh
b5dab5e
@vitaminx
Update gdutilscsinstall.sh
4d7e665
@vitaminx
未格式化
d2089ba
@vitaminx
未格式化
0634d2d
@vitaminx
格式化重新上传
1534cf1
@vitaminx
格式错误
b57da42
@vitaminx
格式错误
1a307e4
@vitaminx
修改换行符错误
833633c
@vitaminx
Delete gdutilsinstall.sh
eb145be
@vitaminx
Delete gdutilsbotinstall.sh
16b5a31
@vitaminx
简化输入！
8eed170 
只需要输入“token”“域名”“tg名”三个就行
@vitaminx
Delete gdutilsbotinstall.sh
cca1d0d
@vitaminx
Delete gdutilsinstall.sh
29c3bff
@vitaminx
更新绑定telegram ID
12945a4 
终于支持绑定telegram ID，这样四个参数都可以判读对错，避免小白输入错误！
@vitaminx
终于支持绑定telegram的id了
f7cc2f1
@vitaminx
一键脚本说明
92f0a96
@vitaminx
Delete 批注 2020-07-02 123957.png
3ce2ad9
@vitaminx
一键脚本说明
29f7aa1
@iwestlin
Copy link
Owner

iwestlin commented Jul 7, 2020

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

@liaojack8
Let users can set the num of button per cols, and the path of SA json…
ad67904 
… files. Fix the mistake in ./src/tg.js 'auth.js'.

Merge remote-tracking branch 'dummy-gdutils/master' into debug
@liaojack8
Copy link
Author

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.

@iwestlin
Copy link
Owner

iwestlin commented Jul 7, 2020

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

@iwestlin
Copy link
Owner

iwestlin commented Jul 7, 2020

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

@liaojack8
Copy link
Author

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

@iwestlin
Copy link
Owner

iwestlin commented Jul 7, 2020

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

@liaojack8
Copy link
Author

已經更新.

@iwestlin
Copy link
Owner

iwestlin commented Jul 7, 2020

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

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

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

@liaojack8
Copy link
Author

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

@vitaminx
Copy link
Contributor

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

@vitaminx
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@liaojack8 @iwestlin @vitaminx