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.

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

Support onlyMimes features, add MimeDetect option of Connector #38

Closed
wants to merge 2 commits into from
Closed

Support onlyMimes features, add MimeDetect option of Connector #38

wants to merge 2 commits into from

Conversation

trannamtrung1st
Copy link
Contributor

  • Connector.cs: add Connector's MimeDetect option, parameters extraction and process request with mimes option

  • IDirectory.cs: change method signature, add list mimes paraemter

  • BaseDriver.cs, IDriver.cs, FileSystemDriver.cs: affected files

  • FileSystemDirectory.cs: implement function: list only files of mimes type if requested.

  • Constants.cs: temp constant values for MimeDectect option


…ion and process request with mimes option

# IDirectory.cs: change method signature, add list mimes paraemter
# BaseDriver.cs, IDriver.cs, FileSystemDriver.cs: affected files
# FileSystemDirectory.cs: implement function: list only files of mimes type if requested.
# Constants.cs: temp constant values for MimeDectect option
---------------------
# BaseModel.cs: fix for failure when uploading non-standard image. issue ref: #36
…the effect --> won't break current code. (I haven't worked with AzureStorage yet)

+ FileSystemController.cs: add demo comment for new feature
+ FileManager/Index.cshtml: add demo comment parameter onlyMimes
@gordon-matt
Copy link
Owner

Thanks for your contribution. This has been implemented. However, I want to give @fsmirne a chance to implement it for the Azure driver as well if he wants to.. before I release the new version

@trannamtrung1st
Copy link
Contributor Author

No problem, I really like this repository so I want to improve it too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants