Skip to content

Update project structure to use submodules for each backend #64

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.

Sign up for GitHub

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

Jump to bottom
Open
zmeda opened this issue Jul 30, 2016 · 3 comments
Open

Update project structure to use submodules for each backend #64

zmeda opened this issue Jul 30, 2016 · 3 comments
Assignees
@zmeda
Labels
enhancement help wanted

Comments

@zmeda
Copy link
Contributor

zmeda commented Jul 30, 2016

As a suggestion from issue #55 we should change project structure accordingly:

  • scala-jsonapi-core - Includes all core functionalities and domain model
  • scala-jsonapi-spray - Spray specific implementation
  • scala-jsonapi-play - Play specific implementation
  • scala-jsonapi-akkahttp - AkkaHttp specific implementation
  • scala-jsonapi-circe - Circe specific implementation

outcome of that should be 4 different artifacts which would all include core as a basis.
zmeda added a commit that referenced this issue Aug 1, 2016
@zmeda
#64: First try to decouple the project into smaller subprojects. NOTE…
45bf132 
…: Code does not compile! Just want to backup changes.
@zmeda zmeda self-assigned this Aug 1, 2016
zmeda added a commit that referenced this issue Aug 3, 2016
@zmeda
#64: Second try to decouple the project into smaller subprojects. NOT…
a1c3408 
…E: Code does not compile! Just want to backup changes.
@zmeda zmeda mentioned this issue Aug 4, 2016
Open
zmeda added a commit that referenced this issue Aug 4, 2016
@zmeda
#64: Third try to decouple the project into smaller subprojects. NOTE…
a0ba2eb 
…: Code does not compile! Just want to backup changes.
@oporkka
Copy link
Collaborator

oporkka commented Feb 1, 2017

@zmeda I fixed some of the more obvious compilation errors #84

@oporkka oporkka mentioned this issue Feb 2, 2017
Merged
zmeda added a commit that referenced this issue Feb 3, 2017
@zmeda
Merge pull request #84 from oporkka/issue/64
c800b66 
#64: Fixed some of test compile errors
zmeda added a commit that referenced this issue Feb 8, 2017
@zmeda
Merge pull request #85 from oporkka/issue/64-merge-current-master
bf6d942 
Issue #64 merge master
@zmeda zmeda mentioned this issue Feb 21, 2017
Open
@oporkka oporkka mentioned this issue Feb 25, 2017
Merged
zmeda added a commit that referenced this issue Feb 28, 2017
@zmeda
Merge pull request #89 from oporkka/issue/64-marshalling
f33f74b 
Issue #64 fix rest of compilation errors
@caleb-ravel
Copy link

LGTM

@oporkka
Copy link
Collaborator

oporkka commented Mar 30, 2017

I'd like to merge this pull request and open related issues about updating the documentation and other possible remaining tasks.

@emanresusername emanresusername mentioned this issue Apr 12, 2017
Open
@oporkka oporkka mentioned this issue Jul 5, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zmeda zmeda

Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@zmeda @oporkka @lasomethingsomething @caleb-ravel