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

Infer format from extension for new entries. #796

Merged
merged 1 commit into from
Dec 6, 2017

Conversation

tech4him1
Copy link
Contributor

- Summary

If a collection has no format specified, we try to infer the format from
the file extension when reading. This commit also allows us to infer the
format from the extension set in the config, so that we can still
create the correct format when making a new file.

Fixes #776.
(Re)based on #795.

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@tech4him1 tech4him1 force-pushed the 776-infer-format-ext branch from 24889df to d6f7d6b Compare December 1, 2017 17:10
@tech4him1 tech4him1 changed the title WIP: Infer format from extension for new entries. Infer format from extension for new entries. Dec 1, 2017
@tech4him1 tech4him1 requested review from Benaiah and erquhart December 1, 2017 17:24
@tech4him1
Copy link
Contributor Author

tech4him1 commented Dec 1, 2017

This should be blocked until 0.8.0.

If a collection has no format specified, we try to infer the format from
the file extension when reading. This commit also allows us to infer the
format from the `extension` set in the config, so that we can still
create the correct format when making a new file.
@erquhart erquhart force-pushed the 776-infer-format-ext branch from d6f7d6b to 799d566 Compare December 6, 2017 14:55
@verythorough
Copy link
Contributor

Deploy preview ready!

Built with commit 799d566

https://deploy-preview-796--netlify-cms-www.netlify.com

@erquhart erquhart merged commit 864083b into master Dec 6, 2017
@erquhart erquhart deleted the 776-infer-format-ext branch December 6, 2017 15:07
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.

Collections with some custom extensions break.
3 participants