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

500 error patch #1

Draft
wants to merge 98 commits into
base: 2.0
Choose a base branch
from
Draft

500 error patch #1

wants to merge 98 commits into from

Conversation

marc2332
Copy link
Member

@marc2332 marc2332 commented Feb 6, 2024

marc2332 and others added 27 commits February 7, 2024 10:52
* Add StateMetadataFeature to bindings, update MetadataFeature in JS, enable anchor state metadata validation

* Update doc comments
* rename wallet data

* remove awaits and update signatures

* make it compile

* rename more wallet data mentions

* unmut wallet

* remove mut

* remove more mut

* update save load test

* clean up

* one more mut

* temporarily disable test

* review

* some nits

* revert backup rename; remove a function

* revert restore rename

* visibility; rename

* set alias

* )

Co-authored-by: Thoralf-M <[email protected]>

* doc

Co-authored-by: Thibault Martinez <[email protected]>

* doc fixes

* fix

* doc 2

Co-authored-by: Thibault Martinez <[email protected]>

---------

Co-authored-by: Thoralf-M <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
* Staking semantic validation

* Staking account creation validation

* Staking feature creation validation

* Feature removal validation

* Transition validation

* Add missing error

* transition reward rule

* Destruction/removal rewards

* StakingBlockIssuerFeatureMissing

* Add new methods
* Add API responses test

* Cleanup

* Add TODOs

* Update import

* Update more TODOs

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Add more ISA tests, clippy

* fmt

---------

Co-authored-by: Thibault Martinez <[email protected]>
* ValidatorsResponse stakers to validators

* TODOs

* python lint
* Add anchorIdToBech32 utils

* Optional

* Remove ?

* Optional

* = None

* sad cat
* rename wallet data

* remove awaits and update signatures

* make it compile

* rename more wallet data mentions

* first impl

* make it compile

* unmut wallet

* remove mut

* remove more mut

* update save load test

* clean up

* one more mut

* refactor

* error handling

* fix tests

* temporarily disable test

* re-include test

* review

* some nits

* revert backup rename; remove a function

* revert restore rename

* visibility; rename

* set alias

* )

Co-authored-by: Thoralf-M <[email protected]>

* doc

Co-authored-by: Thibault Martinez <[email protected]>

* doc fixes

* fix

* get rid of the enum

* little refactor and CI fix

* fix todo

* fix test

* nits

* clean up

* nice up

* only verify ed25519 address at first wallet creation

* review

* fix comment

Co-authored-by: Thibault Martinez <[email protected]>

* suggestions

Co-authored-by: Thibault Martinez <[email protected]>
Co-authored-by: Thoralf-M <[email protected]>

---------

Co-authored-by: Thoralf-M <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
Alex6323 and others added 30 commits March 4, 2024 15:20
* add missing utxo routes

* Core: rename method to GetOutputManaRewards

* align order, doc fixes

* Python: add account congestion workscore, optionals

* NodeJs: fix more routes and align docs

* fix and order exports

* PR suggestions

Co-authored-by: Thoralf-M <[email protected]>

* Core: rm GetNetworkInfo method

* NodeJs: rm GetNetworkInfo method

* Python: rm GetNetworkInfo method

* PR suggestions 2

Co-authored-by: Thoralf-M <[email protected]>

* Python: = None

* format

---------

Co-authored-by: Thoralf-M <[email protected]>
* Add Client::wait_for_transaction_acceptance

* Adapt examples

* Adapt bindings core

* nodejs

* python

* fmt

* review

* Fix URLS

* Remove comments

* fmt
…otaledger#2114)

* Fix implicit account creation address sync for non Ed25519 addresses

* Update sdk/src/wallet/operations/syncing/mod.rs

* Add address check

* Use implicit_account_creation_address

* use implicit_accounts()

---------

Co-authored-by: Thibault Martinez <[email protected]>
Co-authored-by: DaughterOfMars <[email protected]>
* update dependencies

* 0.2.92

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Remove result types and clean up errors

* no_std, wasm

* it was just this one

* nit

* import error

---------

Co-authored-by: Thibault Martinez <[email protected]>
* feat: Reorder wallet events and add a new block event

* Add `BlockSigningHash` and update wallet events tests

* update comment

* add missing unlocks

* I hope this works

* fixes and improvements

* fix typo in nodejs bindings

* emit broadcasting event after block signing and before emitting

* python events

* revert wallet events

* improvements

* use HexEncodedString in ts

* clean up
* little improvements

* add get_validators example with optional pagination

* the example

* fix example doc

* update

* python: add example

* nodejs: add example

* update year

Co-authored-by: Thoralf-M <[email protected]>

* rust: update example

* python: update example

* nodejs: update example

* python: nit

* docs

* fix

* single letter closures

---------

Co-authored-by: Thoralf-M <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
…er#2129)

* Rename inputs_from_transaction_id to get_transaction_inputs

* Update sdk/tests/client/high_level.rs

Co-authored-by: Thibault Martinez <[email protected]>

---------

Co-authored-by: Thibault Martinez <[email protected]>
* block on balance only for non-wasm

* unpub

* move doc

* fix balance and sync_pending_transactions locking

* remove fn

* check for staking

---------

Co-authored-by: Thibault Martinez <[email protected]>
Co-authored-by: Thoralf Müller <[email protected]>
Co-authored-by: Thoralf-M <[email protected]>
* Rename input selection to transaction builder. Move transaction options to client to support a client version of transaction builder.

* fmt

* revert flag name

* find/replace mistakes

* typo

* error swap

* review

* finish renaming

* Fix reward context inputs and tests

* remove context inputs from transaction options

* revert merge mistake

* remove comments

* usused import

* Remove unused ContextInput

* review

* more ts instances

* review

* double copyright

* review

* fix mana allotment not recalculating after changes are made

* remove error variant

* allow providing chains

* DRY

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Add commitment input syntactic checks

* verify_inputs_packable

* Add errors

* remove semantic checks

* Fix tests

* Don't loop if true
* Rename prepare_transaction() to prepare_send_outputs()

* Remove duplicate

* Fix export

* Rename file and move send_outputs()

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Simplify amount requirement

* fix test to match name

* fmt

* println

* sort anchors low

* fmt?

* tuning

* revert comments

* stupid test

* reorder

* Remove anchor

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Remove output `transition_inner`s

* nit

---------

Co-authored-by: DaughterOfMars <[email protected]>
…otaledger#2141)

* Add mana field to transaction builder tests

* Use Build enum in more tests

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Replacey hexPublicKeyToBech32Address by publicKeyHash;
Use Blake2b256Hash instead of ComputeAccountId/ComputeNftId/PublicKeyHash in bindings core

* Fix typo, static instead of async

* Lower case

* Remove __PublicKeyHashMethod__

---------

Co-authored-by: Thibault Martinez <[email protected]>
* Cleanup bindings send methods

* format

* Keep prepare_send

* Revert comment change

---------

Co-authored-by: DaughterOfMars <[email protected]>
… syntactic (iotaledger#2156)

* Make Block Issuer Feature when Staking Feature is present requirement syntactic

* nodejs

* Python

* error

* move things around

* nit
* Fix burn in bindings

* doc

* order
* Remove Bech32ToHex, HexToBech32, AccountIdToBech32, AnchorIdToBech32, NftIdToBech32

* Fix casing

---------

Co-authored-by: Thibault Martinez <[email protected]>
* fix clippy

* nit

---------

Co-authored-by: Thibault Martinez <[email protected]>
* rm ed25519 address generation from wallet

* rename fn

* rm with_address

* update bindings

* handle builder address cases

* unused import

* re-add address to wallet options

* fix tests

* re-enable placeholder test

* move tests

* nodejs: address to wallet options

* revert going through json file for address

* revert unrelated change

* Update sdk/tests/client/secret_manager/address_generation.rs

Co-authored-by: Thoralf-M <[email protected]>

* recipient

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

* bad wildcards

* PR suggestion

* import

* example doc

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

* fix example

* rename generate methods

* Core: add SecretManager:GenerateEd25519AddressAsBech32 method

* update examples and tests

* Core: simplify SecretManager:GenerateEd25519AddressAsBech32 method

* Python: update binding

* nit

* core: undo rename; update docs

* add todo

* Python: nits

* NodeJs: add binding method

* nit

* sdk: undo rename; format

* core: import

* NodeJs: method suffix

* fix test

* nit

* remove single address generation binding

* Python: small fix

---------

Co-authored-by: Thoralf-M <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
Co-authored-by: DaughterOfMars <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
* Remove metrics from InfoResponse

* NetworkMetricsResponse

* get_network_metrics

* Add to bindings core

* python binding

* Nodejs binding

* lint

* python fix

* Comment tests

* lint

* tmp

* TODO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants