Skip to content

Added method to set 'mutable-content' to 1. Fixes #321, #328.#331

Open
cemcatik wants to merge 2 commits intonotnoop:masterfrom
cemcatik:mutable-content
Open

Added method to set 'mutable-content' to 1. Fixes #321, #328.#331
cemcatik wants to merge 2 commits intonotnoop:masterfrom
cemcatik:mutable-content

Conversation

@cemcatik
Copy link
Copy Markdown

No description provided.

@cemcatik
Copy link
Copy Markdown
Author

Any idea how to resolve the timeout error? It's not related to this changelist and I can't repro locally.

Results :

Tests in error: 
  sendOneSimple(com.notnoop.apns.integration.ApnsConnectionTest): test timed out after 2000 milliseconds

Tests run: 112, Failures: 0, Errors: 1, Skipped: 3

return this;
}

/**
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

While this comment below is helpful, I would delete it since it doesn't have anything to setting mutable-content.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Looks like it's just auto-formatted the javadoc of the .method below. I'll clean it up in a bit. Thanks

@grippy
Copy link
Copy Markdown

grippy commented Apr 11, 2017

@cemcatik - One minor issue.

@matzew It looks like you merged the last PR on this repo. Are you still maintaining it? Any idea what we need to do to get this error resolved so we can merge this and publish a new release?

Copy link
Copy Markdown

@grippy grippy left a comment

Choose a reason for hiding this comment

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

Looks good! Hoping this gets merged & released in the near future 😄

@satishbotla
Copy link
Copy Markdown

any one merging this pr?

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.

3 participants