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

non-blockin' SOAP #259

Merged
merged 3 commits into from Jun 5, 2014

Conversation

Projects
None yet
2 participants
@hoff2
Contributor

hoff2 commented Jun 5, 2014

with an sbt setting:

async in scalaxb in Compile := true

hoff2 added some commits May 28, 2014

Modified for asynchronous SOAP
copies of original versions of all changed files kept with _orig prefix
added. All of these changes will next be made conditional/
configurable (original behavior worked back in)
Add async as a config option
Restore original functionality unless config.async is true. Changing
trait names for async versions of the SOAP and Dispatch bits might not
be absolutely necessary but doesn't hurt.

eed3si9n added a commit that referenced this pull request Jun 5, 2014

@eed3si9n eed3si9n merged commit 2ed9954 into eed3si9n:master Jun 5, 2014

@eed3si9n

This comment has been minimized.

Show comment
Hide comment
@eed3si9n

eed3si9n Jun 5, 2014

Owner

Thanks for this change! When @jlavallee sent me #213 I had some vague hope that I'd be making progress into scalaxb2, but that's been stalled for a while, so I'm going to take this one.

Owner

eed3si9n commented Jun 5, 2014

Thanks for this change! When @jlavallee sent me #213 I had some vague hope that I'd be making progress into scalaxb2, but that's been stalled for a while, so I'm going to take this one.

@@ -2,7 +2,7 @@ import Dependencies._
import Common._
val commonSettings = Seq(
version := "1.1.3-SNAPSHOT",
version := "1.1.3-ASYNC",

This comment has been minimized.

@hoff2

hoff2 Jun 5, 2014

Contributor

^^ just noticed this -- I set this so that I could publish it locally and pull it into another project to test some things by specifying this "version" -- may need changed back?

@hoff2

hoff2 Jun 5, 2014

Contributor

^^ just noticed this -- I set this so that I could publish it locally and pull it into another project to test some things by specifying this "version" -- may need changed back?

This comment has been minimized.

@eed3si9n

eed3si9n Jun 5, 2014

Owner

Yea. I'll take care of that.

@eed3si9n

eed3si9n Jun 5, 2014

Owner

Yea. I'll take care of that.

@eed3si9n eed3si9n added this to the 1.2.0 milestone Jun 8, 2014

@eed3si9n eed3si9n added the enhancement label Jun 8, 2014

@eed3si9n

This comment has been minimized.

Show comment
Hide comment
@eed3si9n

eed3si9n Jun 9, 2014

Owner

Posted Future SOAP. Either or throw Exception? to the mailing list. Let me know what you think.

Owner

eed3si9n commented Jun 9, 2014

Posted Future SOAP. Either or throw Exception? to the mailing list. Let me know what you think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment