Conversation
jasonpaulos
approved these changes
Oct 11, 2021
Contributor
jasonpaulos
left a comment
jasonpaulos
left a comment
There was a problem hiding this comment.
Looks good! Once tests pass, this is good to merge
jasonpaulos
added a commit
that referenced
this pull request
Oct 12, 2021
This reverts commit 366c55e.
jasonpaulos
added a commit
that referenced
this pull request
Oct 12, 2021
This reverts commit 2bd5d77.
Aharonee
added a commit
that referenced
this pull request
Jan 20, 2022
(Resolved conflicts) commit ffc8254 Merge: 6e7cbec 7ae8485 Author: egieseke <eric_gieseke@yahoo.com> Date: Thu Dec 30 13:58:19 2021 -0500 Merge branch 'release/1.11.0-beta-2' into develop commit 7ae8485 Author: egieseke <eric_gieseke@yahoo.com> Date: Thu Dec 30 09:59:13 2021 -0500 bump to 1.11.0-beta-2 commit 6e7cbec Author: Hang Su <87964331+ahangsu@users.noreply.github.com> Date: Tue Dec 28 16:38:14 2021 -0500 Support Foreign objects as ABI arguments (#277) * update method/arg/returns/interface/contract definition * ugh no lambda in java7 * update method parsing, bug fix * update * update name checker and test cases * update testcase for method * update testing for contract and interface * update background types * update methods in atomic transaction composer * update methods in atomic transaction composer * update methods in atomic transaction composer, execute and add method call to go * need to update exed * update return value format * put method call option separate * update method, allowing argument to have tx type in string * update method list to map * update clone * update exec * update execute method * update params * yea this should break, add abijson test * update cucumber test for abijson * update cucumber test script * update * yea this one also fail cuz you need atomic transaction composer test in cucumber * there is a bug for transaction argument test, going to fix that * update minor fix to get through test * start working on integrate test for abi * resolve issues partly * working on integration test * make payment trans test in other class * half way through cucumber test * does not go through exec, dunno why * seeing some light ahead, compact the signedTxns with correct way * why returned log is empty? * minor renaming, and removing forced regex method name check * minor changes to rename stuffs and export stuffs * minor * minor * modify transactionSigner to an interface * minor update, update parse error in exec res * minor, return value cannot find abi res, do not throw exception * update sign method to sign interface hash code * update method arg ret with abitype, resolve build error * remove ABIValue and methodArgument * add method call polish * remove redundant lines * dirty fix for sdk not handling log return * update final step check, should pass... * change according to generator * updated naming * update naming * use generator again * minor * minor * resolve issues * start supporting foreign objects * update method test cases * supporting foreign array in atomic transaction composer * update compact foreign array, update array limit check, update method call allowing foreign array * update sender appid/addr to first apat/apas * minor * should be fixed * minor * first commit * update tuple wrapping mechanism, wrap from > 14 to > 15 * update contract appID to networks, should fail * minor * minor renaming * update cucumber test * clean up foreign object array support * minor * minor * minor on group size and group ID * revert test branch, get lost code back * update changes against cucumber test * update SDK testing implementation * resolve part of the review * remove some numerical constants by defining static final variables * minor * minor * optimize index to wait in execute * remove foreign object number constrain * per review * minor for ref type set * minor for contract networks non null * remove wildcard imports * minor * remove optimization on add method call commit 33652fe Author: Hang Su <87964331+ahangsu@users.noreply.github.com> Date: Wed Dec 8 16:39:02 2021 -0500 Upgrade `jackson` packages to resolve PRs on vulnerability update jackson-(core/databind/annotations) to resolve PR #279 and #229 commit b89681a Merge: 1c0e4f4 d398d0f Author: egieseke <eric_gieseke@yahoo.com> Date: Tue Dec 7 16:45:05 2021 -0500 Merge branch 'release/1.11.0-beta-1' into develop commit d398d0f Author: egieseke <eric_gieseke@yahoo.com> Date: Tue Dec 7 15:36:13 2021 -0500 Update Changelog. commit 2855578 Author: egieseke <eric_gieseke@yahoo.com> Date: Tue Dec 7 14:53:02 2021 -0500 Removed PRs for 267, 269, and 270 from Changelog. commit 8aa70c8 Author: egieseke <eric_gieseke@yahoo.com> Date: Tue Dec 7 11:34:00 2021 -0500 bump to 1.11.0-beta-1 commit 1c0e4f4 Author: Hang Su <87964331+ahangsu@users.noreply.github.com> Date: Fri Dec 3 13:49:24 2021 -0500 ABI Interaction Support for JAVA SDK (#268) This PR enables invoking on-chain contract methods using off-chain applications, as defined by ARC-0004. Design doc here: https://gist.github.com/jasonpaulos/a810abe7e86d43840d14445718565a9a commit 53b5583 Author: Hang Su <87964331+ahangsu@users.noreply.github.com> Date: Mon Nov 29 13:09:11 2021 -0500 Bug fix for `logs` on `PendingTransactionResponse` (#275) - update generator for new v2 model in java-sdk - add minor test for PendingTransactionResponse for retrieving logs commit 3086ecd Author: Hang Su <87964331+ahangsu@users.noreply.github.com> Date: Wed Nov 10 18:28:25 2021 -0500 Add WaitForConfirmation function (#274) Export WaitForConfirmation from test to SDK commit cf7248f Author: Will Winder <wwinder.unh@gmail.com> Date: Thu Oct 14 13:10:35 2021 -0400 Better error message on encoding exception. (#258) commit b830635 Author: Jason Paulos <jasonpaulos@users.noreply.github.com> Date: Tue Oct 12 13:29:04 2021 -0700 Revert "Revert "Fix ABI source code position for ABI feature (#260)"" This reverts commit 2bd5d77. commit 605bf4e Author: Jason Paulos <jasonpaulos@users.noreply.github.com> Date: Tue Oct 12 13:28:48 2021 -0700 Revert "Revert "Add ABI encoding support (#255)"" This reverts commit 1debe4e. commit b89da61 Merge: 04b497b cda8a83 Author: Jason Paulos <jasonpaulos@users.noreply.github.com> Date: Tue Oct 12 13:28:25 2021 -0700 Merge branch 'master' into develop commit 04b497b Merge: c488a10 c8f60e9 Author: Brice Rising <brice@algorand.com> Date: Tue Oct 12 15:26:53 2021 -0400 Merge branch 'release/1.10.0' into develop
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change location of ABI source code (and the corresponding location).
Minor updates on doc-strings and protected method inside ABI package.