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

Add MDX type: text/mdx #136

Closed
wants to merge 1 commit into from
Closed

Add MDX type: text/mdx #136

wants to merge 1 commit into from

Conversation

silvenon
Copy link
Contributor

@silvenon silvenon commented Jul 8, 2018

No description provided.

"notes": "JSX in Markdown",
"sources": [
"https://spectrum.chat/thread/1021be59-2738-4511-aceb-c66921050b9a",
"https://github.com/mdx-js/specification"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide a primary source to where this mime is defined. https://github.com/jshttp/mime-db/blob/master/README.md#contributing

I don't see ant mention of the mime type on this page. Perhaps I'm missing it though, if you can point me to were it states the mime and that this file extension it meant to contain the contents for this mime.

@dougwilson
Copy link
Contributor

I know you created an issue upsteam to determine the actual MIME type to use, but it doesn't seem to have any responses. I'm going to close this PR for now since it seems blocked, but if there is movement upstream, we can always open a new one 👍

@dougwilson dougwilson closed this Aug 30, 2018
@silvenon silvenon changed the title Add JSX in Markdown type: text/mdx Add MDX type: text/mdx Sep 18, 2018
@silvenon
Copy link
Contributor Author

silvenon commented Sep 19, 2018

@dougwilson the media type text/mdx is now defined in the official MDX specification.

@dougwilson
Copy link
Contributor

Just open a new PR

@silvenon silvenon mentioned this pull request Apr 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants