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

Rebase 2.0 into master #605

Closed
wants to merge 33 commits into from
Closed

Rebase 2.0 into master #605

wants to merge 33 commits into from

Conversation

JST5000
Copy link
Contributor

@JST5000 JST5000 commented Jun 28, 2023

High Level Overview of Change

Rebase 2.0 onto master

Context of Change

Needed to merge the two branches in preparation for the 2.0 release.

Type of Change

  • Release

Test Plan

CI Passes

connorjchen and others added 30 commits December 1, 2022 14:37
* refactor: Make request_impl into a private function (#478)

* Made request_impl into a private method

* refactor: Alias function for safe_sign_transaction called sign (#470)

* Create alias for safe_sign_transaction

* Update tests to use new submit-transaction alias

* fix: fix Github Actions issues (#485)

* add better debugging

* fix yaml

* switch to ubuntu 20.04

* switch back to latest

* Revert "switch back to latest"

This reverts commit 564002b.

* clean up

* add 3.11 support

Co-authored-by: Mayukha Vadari <mvadari@ripple.com>
* Refactored wallet class and tests

* Fix leading zero for key formatting

* Changed from_entropy to take in hex string

* Update core keypairs docs

* Throw error instead truncate for generate seed with entropy

* Refactor test main wallet generation suite
* Added from secret numbers method and tests

Co-authored-by: Jackson Mills <aim4math@gmail.com>
* Remove deprecated functions

* Fix isort

---------

Co-authored-by: Caleb Kniffen <ckniffen@ripple.com>
* rename AccountSetFlag -> AccountSetAsfFlags

* add AccountSetFlag enum to represent transaction flags

Note: AccountSetAsfFlags are not bit-field flags and therefore
do not require a FlagInterface. At most one asf flag can be
set per transaction.
* Remove send_reliable_submission
* Remove the async sign feature

* Directly alias sign and add comment with history
* Update the CHANGELOG

* Update `autofill_and_sign` and `sign_and_submit`
…#601)

* Remove classic_address from Wallet

* Re-add the `classic_address` alias
* fix XRP.to_amount

* update changelog

* fix circular dependency

* fix circular dependency

* add comment

* fix test

* improve comment
* refactor: Make request_impl into a private function (#478)

* Made request_impl into a private method

* refactor: Alias function for safe_sign_transaction called sign (#470)

* Create alias for safe_sign_transaction

* Update tests to use new submit-transaction alias

* fix: fix Github Actions issues (#485)

* add better debugging

* fix yaml

* switch to ubuntu 20.04

* switch back to latest

* Revert "switch back to latest"

This reverts commit 564002b.

* clean up

* add 3.11 support

Co-authored-by: Mayukha Vadari <mvadari@ripple.com>
* Refactored wallet class and tests

* Fix leading zero for key formatting

* Changed from_entropy to take in hex string

* Update core keypairs docs

* Throw error instead truncate for generate seed with entropy

* Refactor test main wallet generation suite
* Added from secret numbers method and tests

Co-authored-by: Jackson Mills <aim4math@gmail.com>
* Remove deprecated functions

* Fix isort

---------

Co-authored-by: Caleb Kniffen <ckniffen@ripple.com>
* rename AccountSetFlag -> AccountSetAsfFlags

* add AccountSetFlag enum to represent transaction flags

Note: AccountSetAsfFlags are not bit-field flags and therefore
do not require a FlagInterface. At most one asf flag can be
set per transaction.
* Remove send_reliable_submission
* Remove the async sign feature

* Directly alias sign and add comment with history
* Update the CHANGELOG

* Update `autofill_and_sign` and `sign_and_submit`
JST5000 and others added 3 commits June 28, 2023 12:36
…#601)

* Remove classic_address from Wallet

* Re-add the `classic_address` alias
* fix XRP.to_amount

* update changelog

* fix circular dependency

* fix circular dependency

* add comment

* fix test

* improve comment
@mvadari mvadari changed the base branch from xrpl-py-2.0 to master June 28, 2023 21:04
@JST5000
Copy link
Contributor Author

JST5000 commented Jun 28, 2023

Superceded by #606

@JST5000 JST5000 closed this Jun 28, 2023
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.

4 participants