Skip to content

Fix #636 #688

Open
riya24899 wants to merge 4 commits intoprotegeproject:masterfrom
riya24899:master
Open

Fix #636 #688
riya24899 wants to merge 4 commits intoprotegeproject:masterfrom
riya24899:master

Conversation

@riya24899
Copy link
Copy Markdown

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
Copy Markdown

@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
Copy Markdown
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?

Comment thread webprotege-server/pom.xml
<groupId>net.sourceforge.owlapi</groupId>
<artifactId>owlapi-gwt</artifactId>
<version>4.3.2.1-SNAPSHOT</version>
<version>4.3.2.1</version>
Copy link
Copy Markdown
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.

Comment thread webprotege-client/pom.xml
<groupId>net.sourceforge.owlapi</groupId>
<artifactId>owlapi-gwt</artifactId>
<version>4.3.2.1-SNAPSHOT</version>
<version>4.3.2.1</version>
Copy link
Copy Markdown
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
Copy Markdown
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