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.
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
Regexp for 'documents' should also support docx #790
Comments
Isn't it rather simple to add new doctypes? So why should we bloat the default? (Most if the time I even reduce it to only accept PDF.) |
I agree Michael. I can see why adding to the default would be nice, to avoid repetitive typing, but it's not a big requirement IMO. |
Because doc is now esdentially deprecated and has been superseded by docx. I don't consider this bloat at all...it makes it more useful. Smart defaults make for an easier life. |
Not a big requirement but I think a requirement nevertheless. |
Well, OK then! If you like it, let's do it. |
Pull request sent. Included in this commit. |
Now the regexp for 'documents' is:
but this should be:
(perhaps there are also some more filetypes to add to this regexp?
The text was updated successfully, but these errors were encountered: