Skip to content
This repository has been archived by the owner on Nov 28, 2023. It is now read-only.

选择文件上传可在前端验证文件后缀，但拖拽上传还无法验证，待解决。 #561

Merged
merged 3 commits into from
Sep 7, 2017
Merged

选择文件上传可在前端验证文件后缀，但拖拽上传还无法验证，待解决。 #561

merged 3 commits into from
Sep 7, 2017

Conversation

PSoul
Copy link
Contributor

@PSoul PSoul commented Sep 7, 2017

No description provided.

@coveralls
Copy link

coveralls commented Sep 7, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 41.55% when pulling a46c855 on PSoul:master into 2d979c5 on wufeifei:develop.

@FeeiCN
Copy link
Owner

FeeiCN commented Sep 7, 2017

@PSoul 拖拽上传为什么无法测试了？

@PSoul
Copy link
Contributor Author

PSoul commented Sep 7, 2017

@wufeifei 不是说无法测试，现在这个代码可以对手动以及拖拽上传的后缀名做检测，但是无法禁止文件夹上传，如果文件夹里存在大量的不合法后缀的话，拖拽文件夹上去一样会弹出大量的框，解决方案是禁用文件夹上传。我在看怎么搞。。

@FeeiCN
Copy link
Owner

FeeiCN commented Sep 7, 2017

@PSoul 好

@coveralls
Copy link

coveralls commented Sep 7, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.04%) to 41.585% when pulling 9f9646d on PSoul:master into 2d979c5 on wufeifei:develop.

@coveralls
Copy link

coveralls commented Sep 7, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.9%) to 42.428% when pulling ca657b5 on PSoul:master into 2d979c5 on wufeifei:develop.

@PSoul
Copy link
Contributor Author

PSoul commented Sep 7, 2017

@wufeifei fix #542 ，在第二个commit里搞定了，有空去验证下，OK的话就关了issue吧

@FeeiCN
Copy link
Owner

FeeiCN commented Sep 7, 2017

@PSoul ok, thx！

@FeeiCN FeeiCN merged commit e88ba4b into FeeiCN:develop Sep 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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
@PSoul @coveralls @FeeiCN