Skip to content

Conversation

@riya24899
Copy link

Add functionality to export a project as an OBO file.

riya24899 and others added 4 commits May 23, 2020 18:48
Add support for OBO files
Adding functionality for OBO files
DownloadFormatExtension fixed, OBO supported, XML files fixed
@riya24899 riya24899 changed the title Fix #636 , #642 Fix #636 May 25, 2020
@VishalS711
Copy link

@riya24899 and I were working together on this issue. Here are some screenshots of the added OBO download feature.

Screenshot from 2020-05-29 09-41-09
Screenshot from 2020-05-29 09-41-26


OWL_XML(OWLXMLDocumentFormat::new, "application/owl+xml", DownloadFormatExtension.owx),

OBO(OBODocumentFormat::new, "application/octet-stream", DownloadFormatExtension.obo),
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious. Where does this Mime Type come from?

<groupId>net.sourceforge.owlapi</groupId>
<artifactId>owlapi-gwt</artifactId>
<version>4.3.2.1-SNAPSHOT</version>
<version>4.3.2.1</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please can you remove this? It's since been updated.

<groupId>net.sourceforge.owlapi</groupId>
<artifactId>owlapi-gwt</artifactId>
<version>4.3.2.1-SNAPSHOT</version>
<version>4.3.2.1</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please can you remove this? It's since been updated.

Copy link
Contributor

@matthewhorridge matthewhorridge left a comment

Choose a reason for hiding this comment

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

Looks good. I've just a couple of minor comments. Easy to fix. Thanks a lot for the contribution.

@matthewhorridge matthewhorridge added Type: Enhancement A request for a change that is an enhancement Status: Needs Review Indicates that a code change, in a branch/PR, needs to be reviewed labels May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Needs Review Indicates that a code change, in a branch/PR, needs to be reviewed Type: Enhancement A request for a change that is an enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants