Conversation
This PR is kinda "Work In Progress". Testing it using one of our 2.5 projects during the day. Feel free to give feedback. |
Should you set build.properties: sbt.version = 0.13.11 (?) |
@andersha Thx. Done... |
I did these changes locally, any reason for keeping cross-compile? -scalaVersion:= "2.11.6" |
@andersha IMHO not. Would be great to know the initial intention, why |
Have to rewrite the Scanner... I've opened a question on StackOverflow to get information how routes should be pulled out of Play! 2.5 the correct way: |
I'm interested in having this resolved - are there any other issues besides the Routes Scanner? I asked Play developers about this issue, and they said the routes shouldn't have changed, and the issue should be filed as a bug against Play project. |
I got an error using this branch:
The cause is that an old version of
|
Also, everything appears to work correctly at first glance, the only change I had to make is adding |
Any ETA on this? We are very interested in pulling this in as well... |
play.modules.swagger.SwaggerApplicationLoader is to support Compile Time DI, so it should not be removed. |
|
Will have a look at the input from @danosipov and @YannMoisan tomorrow... |
@frne how is this going? |
how to fix it(#68)? |
Tried using this branch with some new Swagger notations, cleared 2 exceptions but I'm still getting this exception thrown: play.api.UnexpectedException: Unexpected exception[CreationException: Unable to create injector, see the following errors:
|
I get the same exception as @rney-aspect mentioned. Has anyone solved this yet? |
Did anyone fix that? |
Things are getting confusing... Maybe I'll find some time during the extended weekend. Planning to integrate the feedback form @danosipov. @rney-aspect could you post the full exception stacktrace? |
I'm getting this:
|
@shijinkui i believe this PR is still a WIP. When @frne feels good about it and the issues are cleared, we can merge. |
I am stiil getting the error. Any fix for this and which version?
3 errors |
As already mentioned by @fehguy, this is a "work in progress" PR, which means: @gauravranjan11 No there is no fix yet. AFAIK, there is no working version with Play 2.5. Sorry, but I have really not much time these days. Generally said, I'm working on it, but cannot estimate any finish or merge date. |
I have the feeling, we got off topic here... So...when will it be done? |
may be we should revert to our trusted companion spring. On Thu, May 19, 2016 at 6:20 PM, verysoon notifications@github.com wrote:
Regards, |
@shijinkui I ended up using this pull request instead: I have it working fine in Play 2.5 |
Closed in favor of #76 |
good job!@frne |
im still facing issues in #76 On Fri, May 20, 2016 at 12:21 PM, tommy notifications@github.com wrote:
Regards, |
libraryDependencies is "io.swagger" %% "swagger-play2" % "1.5.3-SNAPSHOT" ? @gauravranjan11 |
works for me with play 2.5.3 if i fix jackson-module-scala dependency |
How to fix jackson module dependency?
|
@tommywu23 im getting following error sbt.ResolveException: unresolved dependency: io.swagger#swagger-play2_2.11;1.5.3-SNAPSHOT: not found |
For people that are googling for a Play 2.5 compatible package and end up on this page, I created this temporary repo that should work while we wait for official support to be added to swagger-play: |
Fixes incompatibilities with Play Framework 2.5.x
play.modules.swagger.SwaggerApplicationLoader
in favour of DI (uses the removed Play Plugin API)Probably fixes #66 and #67
ToDo:
java.lang.AbstractMethodError
inApiHelpController.scala:68