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

java transfer code examples #339

Merged
merged 2 commits into from
Feb 7, 2020
Merged

Conversation

dgarcia360
Copy link
Collaborator

Continuation of #101

@dgarcia360 dgarcia360 requested a review from fboucquez February 3, 2020 17:54
@dgarcia360 dgarcia360 changed the title Added transfer commands java transfer code examples Feb 3, 2020
@dgarcia360 dgarcia360 requested a review from evias February 4, 2020 11:45
Copy link
Contributor

@fboucquez fboucquez left a comment

Choose a reason for hiding this comment

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

Transfer java examples can be merged back to the java samples branch.

Once we have the recommended max fee, I'll add the recommended value to the SDK and the examples should be updated using that value when creating transactions.

@dgarcia360 dgarcia360 merged commit 5781a6c into f2-java-samples Feb 7, 2020
dgarcia360 added a commit that referenced this pull request Feb 7, 2020
* Initial migration of existing examples to the new nem2 java sdk
Upgraded and simplified gradle build

* Added skeleton

* Added account java guides

* Added ConvertingAnAccountToMultisig

* Upgrade sdk to latest 0.15.0
Simplified how networkType and generationHash is loaded

* Added java get examples

* Fixed example

* Updated testnet endpoints

* java  transfer code examples (#339)

* Added transfer commands

* Removed hardcoded NetworkType for repository loaded one
Small build.gradle improvement

Co-authored-by: fboucquez <[email protected]>

* Removed hardcoded NetworkType

Co-authored-by: fboucquez <[email protected]>
@dgarcia360 dgarcia360 deleted the java-examples-transfer branch February 7, 2020 17:54
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.

2 participants