mm-wiki
mm-wiki copied to clipboard
加入图片粘贴上传功能
@wolcengit 首先感谢提交 PR,可以按功能分开提多个 PR. 按功能合并。
- 现在上传图片是需要保存到数据库,图片粘贴是否能保证上传直接到服务器
- LDAP 功能我也一直在考虑是否要加。之前之所以不加 LDAP 的原因如下:
- 很多小公司或个人没域帐号,但是有自己简单的员工管理
- 如果加了 LDAP 域账号,是否要支持其他的域账号登录?
所以使用了简单的统一的思想,简单的开发一个登录接口和 MM-wiki 对接。
如果要加 LDAP,如果满足没有 LDAP 域账号的那一部分用户的需求。或者说加入 LDAP和现有的统一登录如何结合起来?
在PR时候后面几个功能还没有提交,为啥全部挂过来显示了,望指点下。 我目前的做法是简化了登录界面,内部处理还是使用现有的统一登录,只是做了拓展,支持ldap和http两个协议对接。
@wolcengit 创建提多个PR。
@wolcengit 之所以都挂过来了是因为pr的本质是将请求合并某一分支,而不是某一次提交。如果在创建pr之后在该分支上有后续commit,在merge之前这些commit也都会挂过来,这时如果进行merge,则后续的commit也都会被merge过来了。
@cifaz
@wolcengit 最近这个版本要加 ldap,能否将之前的提的 ldap 那部分单独提一个 pr 合并过来。看实现是没有问题的。更改了用户的统一登录前缀这里可以再讨论下。用户登录时还要加 @ 符有点奇怪。如果可以,请加开发者QQ交流群,我们讨论下:922827699。备注下
上传文件不支持批量上传,希望支持批量上传文档
这么好的功能居然不合并进来,我先在本地加上了