-
Notifications
You must be signed in to change notification settings - Fork 205
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
3.0.0: Sync changes to develop #884
Merged
Merged
Conversation
This file contains 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
* Remove buffer usage in favor or Uint8Array and Dataview * Add byteoffset when creating DataView * Increase Indexer sleep time * Add polyfill for base64 decoding and update examples * Update examples to use utility function for Buffer functionality * Revert smoke test * Add a roundtrip test for sanity check * Fix base64 encoding test * Remove unnecessary casting * Unify return type to Uint8Array * Standardize tests and add more utf-8 strings * Add hex encoding and string encoding tests * Update tests for edge bytes * Export byte conversion utilities * Fix imports to prefix algosdk * Fix remaining imports in examples * Add test case for non UTF8 characters
* Update Chromedriver link 115>= * Use puppeteer to download latest stable chromedriver * Update circle ci browser tools orb * Don't manually install chromedriver * Pin chromedriver version * Remove comment * Revert to 114 * Update selenium * Test * Update chromedriver 114
* Remove buffer usage in favor or Uint8Array and Dataview * Add byteoffset when creating DataView * Increase Indexer sleep time * Add polyfill for base64 decoding and update examples * Update examples to use utility function for Buffer functionality * Revert smoke test * Add a roundtrip test for sanity check * Fix base64 encoding test * Remove buffer in transaction * Remove unnecessary casting * Unify return type to Uint8Array * Remove buffer usage in source code except client * Standardize tests and add more utf-8 strings * Add hex encoding and string encoding tests * Fix some code broken by existing tests * Update tests for edge bytes * Export byte conversion utilities * Fix imports to prefix algosdk * Resolving merge conflicts * Remove some generated types from stable release * Fix node tests * Try loadResource in browser * Update buffer usage in client files * Revert cucumber browser loadResource * Fix browser tests * Fix node tests * Add alternative path for decoding composer responses * Remove browser buffer dependency, fix cucumber tests * Revert cucumber node index.js file to use buffer again * Fix some conversion methods in err messages * Update src/encoding/binarydata.ts Co-authored-by: Eric Warehime <eric.warehime@gmail.com> * Run prettier on commit suggestion * Add toString conversion for buffers * Update tests/cucumber/browser/test.js Co-authored-by: Jason Paulos <jasonpaulos@users.noreply.github.com> --------- Co-authored-by: Eric Warehime <eric.warehime@gmail.com> Co-authored-by: Jason Paulos <jasonpaulos@users.noreply.github.com>
* Add address bytes length check in encodeAddress * Fix example where address is invalid * Add unit test for incorrect length encoding
Merge develop into 3.0.0
Co-authored-by: Jason Paulos <jasonpaulos@users.noreply.github.com>
…types in models (#852)
gmalouf
approved these changes
Aug 20, 2024
This was referenced Aug 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Development on v3 of the JS SDK has so far been isolated to the
3.0.0
branch. This PR merges the new changes back into thedevelop
branch.