-
Notifications
You must be signed in to change notification settings - Fork 15
fix: Update aleph.im & twentysix to aleph.cloud #414
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
Merged
Merged
Conversation
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
|
Failed to retrieve llama text: POST 504: Gateway Timeout |
odesenfans
approved these changes
Nov 3, 2025
Contributor
odesenfans
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
nesitor
added a commit
that referenced
this pull request
Nov 27, 2025
* Problem: Ledger wallet users cannot use Aleph to send transactions. Solution: Implement Ledger use on CLI to allow using them. Do it importing a specific branch of the SDK. * Fix: Solve code quality issues. * Fix: Solve issue loading the good configuration. * Fix: Solved definitively the wallet selection issue and also solved another issue fetching instances from scheduler. * Fix: Solved code-quality issue. * Feature: cli load_account to handle account selections * Fix: using load_account instead of _load_account * fix: unit test mock * Feature: aleph account init to create based config for new user using ledger * fix: aleph account list now handle ledger device * fix: aleph account address now handle ledger device * fix: aleph account export-private-key handle ledger case (can't export private key) * Feature: missing unit test for ledger * fix: Update aleph.im & twentysix to aleph.cloud (#414) Update broken doc links and references to old brand names (aleph.im, twentysix.cloud) to aleph.cloud * Fix: handle common error using ledger (OsError / LedgerError) * Fix: handle change from account on sdk side * Fix: remove init commands and ensure that config file/folder and subfolder are created * Fix: AccountLike renamed to AccountTypes * fix: AlephAccount should bez AccountTypes * fix: account init commands unit test should be removed since not usefull anymore * Fix: use arguments instead of get_closest_tier (#416) * Fix: use arguments instead of get_closest_tier * Fix: ensure that hold can't use --vcpus or --memory or --rootfs_size upper than tier limit * fix: wrong display of vcpus on pricing display * Fix: instance create crn is none when when giving --crn-hash or --crn… (#417) * Fix: instance create crn is none when when giving --crn-hash or --crn-url * fix: linting * Feature: utils functions for ledger * Fix: ensure ledger is connected before loading ledger account * Fix: avoid connecting to ledger when not needed * Fix: use BaseEthAccount instead of EthAccount in instance create and prefetch crn list * fix: refactor aleph account configure and list to handle ledger * Fix: call_program_crn_list can now filter node when fetching if they active or not * Fix: unit test * Feature: --no args for aleph account configure * fix: linting issue * Feature: load acount unit test * Feature: ledger can be load from derivation path * Unit: test_aggregate.py for ledger * Unit: new tests for utils func around ledger (wait_for_ledger_connection, ...) * Unit: new tests for non interactive account config * Problem: Ledger wallet users cannot use Aleph to send transactions. Solution: Implement Ledger use on CLI to allow using them. Do it importing a specific branch of the SDK. * Fix: Solve code quality issues. * Fix: Solve issue loading the good configuration. * Fix: Solved definitively the wallet selection issue and also solved another issue fetching instances from scheduler. * Fix: Solved code-quality issue. * Feature: cli load_account to handle account selections * Fix: using load_account instead of _load_account * fix: unit test mock * Feature: aleph account init to create based config for new user using ledger * fix: aleph account list now handle ledger device * fix: aleph account address now handle ledger device * fix: aleph account export-private-key handle ledger case (can't export private key) * Feature: missing unit test for ledger * Fix: handle common error using ledger (OsError / LedgerError) * Fix: handle change from account on sdk side * Fix: remove init commands and ensure that config file/folder and subfolder are created * Fix: AccountLike renamed to AccountTypes * fix: AlephAccount should bez AccountTypes * fix: account init commands unit test should be removed since not usefull anymore * Feature: utils functions for ledger * Fix: ensure ledger is connected before loading ledger account * Fix: avoid connecting to ledger when not needed * Fix: use BaseEthAccount instead of EthAccount in instance create and prefetch crn list * fix: refactor aleph account configure and list to handle ledger * Fix: call_program_crn_list can now filter node when fetching if they active or not * Fix: unit test * Feature: --no args for aleph account configure * fix: linting issue * Feature: load acount unit test * Feature: ledger can be load from derivation path * Unit: test_aggregate.py for ledger * Unit: new tests for utils func around ledger (wait_for_ledger_connection, ...) * Unit: new tests for non interactive account config * fix: allow user to specify how many ledger account they want to load (default: 5) * fix: use the already existing args chain to load_account * fix: lint issue * Refactor: remove code duplication * Fix: aggregate unit test * Fix: non-evm account couldn't be used using --private-key or --private-key-file * fix: rename --no of configure commands to --non-it * fix: non interactive args for aleph account config is --non-it not --no * Fix: allow user to increase number of ledger account fetch using aleph account configure --ledger-count --------- Co-authored-by: Andres D. Molins <nesitor@gmail.com> Co-authored-by: 1yam <lyam.gomes@epitech.eu> Co-authored-by: Reza Rahemtola <49811529+RezaRahemtola@users.noreply.github.com> Co-authored-by: 1yam <40899431+1yam@users.noreply.github.com>
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.
Update broken doc links and references to old brand names (aleph.im, twentysix.cloud) to aleph.cloud