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

fix (webapi): add searchbox gui element to hasOtherSomething property #828

Merged
merged 2 commits into from
Apr 17, 2018

Conversation

musicEnfanthen
Copy link
Contributor

@musicEnfanthen musicEnfanthen commented Apr 17, 2018

This PR fixes the missing searchbox gui element for hasOtherSomething property in the something ontology.

fixes #827

@musicEnfanthen
Copy link
Contributor Author

musicEnfanthen commented Apr 17, 2018

Ready to review.

knora-base:objectClassConstraint knora-base:LinkValue .
knora-base:objectClassConstraint knora-base:LinkValue ;

salsah-gui:guiElement salsah-gui:Searchbox .
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add a newline at the end of the file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry I missed that. Added the new line right now.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks! It shouldn't matter, the triplestore did not complain ;-) It is rather a issue of style.

@dasch-swiss dasch-swiss deleted a comment from musicEnfanthen Apr 17, 2018
@tobiasschweizer
Copy link
Contributor

I ran the browser tests for salsah1 and they passed :-)

see #829

Copy link
Contributor

@tobiasschweizer tobiasschweizer left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the fix!

@musicEnfanthen musicEnfanthen merged commit 2c14a10 into develop Apr 17, 2018
@musicEnfanthen musicEnfanthen deleted the fix/has-other-somethin-link branch April 17, 2018 09:35
SepidehAlassi added a commit that referenced this pull request Apr 30, 2018
* develop:
  feature (webapi): make project shortcode required (#824)
  doc (webapi): jekyll / paradox based documentation website (#751)
  Validate ontologies loaded from the triplestore (#806)
  fix (webapi): add searchbox gui element to hasOtherSomething property (#828)
  feature (webapi): allow loading data into graphdb on windows (internal PR) (#814)
  build (webapi): add missing library (#822)
  feature (webapi): allow changing of user’s password by system admin (#815)
  feature (webapi): OpenAPI / Swagger API documentation generation (#812)
SepidehAlassi added a commit that referenced this pull request Apr 30, 2018
* develop:
  feature (webapi): make project shortcode required (#824)
  doc (webapi): jekyll / paradox based documentation website (#751)
  Validate ontologies loaded from the triplestore (#806)
  fix (webapi): add searchbox gui element to hasOtherSomething property (#828)
  feature (webapi): allow loading data into graphdb on windows (internal PR) (#814)
  build (webapi): add missing library (#822)
  feature (webapi): allow changing of user’s password by system admin (#815)
  feature (webapi): OpenAPI / Swagger API documentation generation (#812)
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.

SALSAH V1 GUI: has other something
2 participants