Skip to content

Update scrypt allowed range as defined in pjs repo#2158

Open
paulormart wants to merge 1 commit intoparitytech:masterfrom
turboflakes:master
Open

Update scrypt allowed range as defined in pjs repo#2158
paulormart wants to merge 1 commit intoparitytech:masterfrom
turboflakes:master

Conversation

@paulormart
Copy link
Contributor

Hello this change goes in line with what is currently defined in PJS repo here which allows to load/decrypt recent generated accounts.

cargo test test_get_ --features polkadot-js-compat -- --nocapture

running 2 tests
test polkadot_js_compat::test::test_get_alice_keypair_sr25519 ... ok
test polkadot_js_compat::test::test_get_bob_keypair_sr25519 ... ok

test result: ok. 2 passed; 0 failed; 0 ignored; 0 measured; 12 filtered out; finished in 19.10s

@paulormart paulormart requested a review from a team as a code owner January 20, 2026 13:51
Copy link
Collaborator

@jsdw jsdw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@jsdw
Copy link
Collaborator

jsdw commented Jan 22, 2026

Let's wait for the big v0.50.0 PR to merge which fixes clippy lints and generally changes a bunch, and then merge this on top. I don't want us to start changing more files here because it'll just introduce conflicts with that rather big change.

@jsdw
Copy link
Collaborator

jsdw commented Feb 2, 2026

Let's wait for the big v0.50.0 PR to merge which fixes clippy lints and generally changes a bunch, and then merge this on top. I don't want us to start changing more files here because it'll just introduce conflicts with that rather big change.

Just FYI this is done now and so this PR can be rebased as needed shouldn't be a problem to merge :)

@jsdw
Copy link
Collaborator

jsdw commented Feb 13, 2026

@paulormart could you merge master; that will hopefully fix the error and then I can merge this :)

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.

2 participants