Skip to content

feat(1540): Replace outdated key schemas with keySchema#1547

Open
matevszm wants to merge 1 commit intomainfrom
feat-1540/replace-outdated-key-schemas
Open

feat(1540): Replace outdated key schemas with keySchema#1547
matevszm wants to merge 1 commit intomainfrom
feat-1540/replace-outdated-key-schemas

Conversation

@matevszm
Copy link
Contributor

@matevszm matevszm requested a review from a team as a code owner March 10, 2026 09:41
@github-actions
Copy link

github-actions bot commented Mar 10, 2026

Unit Test Results

889 tests  ±0   889 ✅ ±0   1m 52s ⏱️ -2s
 94 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit ad2be1d. ± Comparison against base commit 33dc43f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Mar 10, 2026

Integration Test Results

36 tests  ±0   36 ✅ ±0   5m 20s ⏱️ ±0s
21 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit ad2be1d. ± Comparison against base commit 33dc43f.

♻️ This comment has been updated with latest results.

@matevszm matevszm force-pushed the feat-1540/replace-outdated-key-schemas branch from 09127c4 to 2ac2971 Compare March 10, 2026 10:09
Copy link
Contributor

@mmyslblocky mmyslblocky left a comment

Choose a reason for hiding this comment

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

adjust description for KeySchema in every place

token: EntityReferenceSchema.describe('Token identifier (ID or name)'),
account: PrivateKeyWithAccountIdSchema.describe(
account: KeySchema.describe(
'Account to associate. Can be {accountId}:{privateKey pair}, key reference or account alias.',
Copy link
Contributor

Choose a reason for hiding this comment

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

change description so that we can use any format of the key

),
adminKey: PrivateKeySchema.optional().describe(
adminKey: KeySchema.optional().describe(
'Admin key of token. Can be {accountId}:{privateKey} pair, account private key in {ed25519|ecdsa}:private:{private-key} format, key reference or account alias. Defaults to operator key.',
Copy link
Contributor

Choose a reason for hiding this comment

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

change description so that we can use any format of the key

),
adminKey: PrivateKeySchema.optional().describe(
adminKey: KeySchema.optional().describe(
'Admin key of token. Can be {accountId}:{privateKey} pair, account private key in {ed25519|ecdsa}:private:{private-key} format, key reference or account alias. Defaults to operator key.',
Copy link
Contributor

Choose a reason for hiding this comment

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

change description so that we can use any format of the key

),
supplyKey: PrivateKeySchema.describe(
supplyKey: KeySchema.describe(
'Supply key. Can be {accountId}:{privateKey} pair, account private key in {ed25519|ecdsa}:private:{private-key} format, key reference or account alias.',
Copy link
Contributor

Choose a reason for hiding this comment

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

change description so that we can use any format of the key

@matevszm matevszm force-pushed the feat-1540/replace-outdated-key-schemas branch from 2ac2971 to d577320 Compare March 12, 2026 13:59
Signed-off-by: matevszm <mateusz.marcinkowski@blockydevs.com>
@matevszm matevszm force-pushed the feat-1540/replace-outdated-key-schemas branch from d577320 to ad2be1d Compare March 12, 2026 14:18
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