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

feat(api-v2): Remove client code generation #1610

Merged
merged 8 commits into from Feb 28, 2020
Merged

Conversation

benjamingeer
Copy link

@benjamingeer benjamingeer commented Feb 25, 2020

  • Remove client code generation (leaving client test data generation).
  • Remove external schema of admin ontology.
  • Update docs.
  • Remove client test data dependency on incunabula.
  • Change JsonLDUtilSpec so it doesn't start an ActorSystem.

Resolves #1607.
Resolves #1600.

@benjamingeer benjamingeer added API/V2 clientapi frontend: Salsah, DSP-APP, BEOL, MLS, etc. labels Feb 25, 2020
@benjamingeer benjamingeer added this to the 2020.1 milestone Feb 25, 2020
@benjamingeer benjamingeer self-assigned this Feb 25, 2020
@benjamingeer
Copy link
Author

@tobiasschweizer I don't know what needs to be changed in knora-api-js-lib for this, could you possibly help? I can probably do some of it if you can give me a list of things to do.

@tobiasschweizer
Copy link
Contributor

This PR will surely affect the way the generated test data is integrated into knora-api-js-lib.

The integration scripts for the admin part will have to be adapted (see also https://github.com/dasch-swiss/knora-api-js-lib/blob/master/package.json and https://github.com/dasch-swiss/knora-api-js-lib/blob/master/Makefile). Also E2E test for the admin part will have to be written. For now, there are only unit tests for the admin part.

I can help you with that, but only after my holiday. I will be back to work on Monday, 9 March.

@benjamingeer
Copy link
Author

I can help you with that, but only after my holiday. I will be back to work on Monday, 9 March.

OK no problem, let's do it then.

@benjamingeer benjamingeer mentioned this pull request Feb 27, 2020
@benjamingeer
Copy link
Author

@subotic The JS Lib Tests are failing, do you know what I would need to change to make them work again? Or does this need help from Tobias?

Benjamin Geer added 2 commits February 27, 2020 13:24
# Conflicts:
#	webapi/src/main/scala/org/knora/webapi/routing/v2/ValuesRouteV2.scala
#	webapi/src/main/scala/org/knora/webapi/util/StringFormatter.scala
@benjamingeer
Copy link
Author

@subotic Do you think this could be merged even though the JS Lib tests are failing? Then I wouldn't have to wait until Tobias gets back from holiday.

@subotic
Copy link
Collaborator

subotic commented Feb 28, 2020

Yeah sure.

Copy link
Collaborator

@subotic subotic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks :-) I will take a look at knora-api-js-lib. Maybe I can fix it.

@benjamingeer
Copy link
Author

Thanks for the review!

@benjamingeer benjamingeer merged commit 6977ab3 into develop Feb 28, 2020
@benjamingeer benjamingeer deleted the wip/1607-client-code branch February 28, 2020 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API/V2 clientapi frontend: Salsah, DSP-APP, BEOL, MLS, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove client code generation Make client tests not depend on incunabula
3 participants