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 MIME type #332

Merged
merged 2 commits into from
Jul 12, 2018
Merged

Add MIME type #332

merged 2 commits into from
Jul 12, 2018

Conversation

Partmedia
Copy link
Contributor

Add model/slvs MIME type, which allows .slvs files to be opened from FreeDesktop-compliant graphical environments.

@whitequark
Copy link
Contributor

Thank you for your contribution, and welcome. The SolveSpace project has a Contributor License Agreement; in order to merge your pull request, please sign it at https://cla-assistant.io/solvespace/solvespace. It will take only a small amount of your time.

@Partmedia
Copy link
Contributor Author

I've agreed to the CLA.

@@ -3,6 +3,7 @@ Version=1.0
Name=SolveSpace
Comment=A parametric 2d/3d CAD
Exec=${CMAKE_INSTALL_FULL_BINDIR}/solvespace
MimeType=model/slvs
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be something like application/x-solvespace? I think all MIME types not explicitly assigned by IANA should be in the x- namespace.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right. I'll make the change.

@Partmedia
Copy link
Contributor Author

Fixed. If it's satisfactory I can squash and rebase.

@whitequark whitequark merged commit 572869d into solvespace:master Jul 12, 2018
@whitequark
Copy link
Contributor

Thank you again!

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.

None yet

2 participants