-
Notifications
You must be signed in to change notification settings - Fork 63
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
chore(hardware-trezor): refactor TrezorKeyAgent to use Transform interface #1096
chore(hardware-trezor): refactor TrezorKeyAgent to use Transform interface #1096
Conversation
357e3f4
to
beda310
Compare
Standard DiffPostThis PR would generate the following Previewdiff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-backend /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-backend
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-backend 2024-02-14 13:21:51.329566527 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-backend 2024-02-14 13:21:51.329566527 +0000
@@ -4,11 +4,11 @@
annotations:
deployment.kubernetes.io/revision: "8"
creationTimestamp: "2024-01-25T22:08:02Z"
- generation: 8
+ generation: 9
labels:
app: backend
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-backend
@@ -89,7 +89,7 @@
value: http://dev-preview-cardano-stack-metadata.dev-preview.svc.cluster.local
- name: USE_KORA_LABS
value: "true"
- image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:2a9qvpsiryyfpp3y5q5zx918zb31gym2
+ image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:ckifi60diw14lzqh5d282y0i2fc79l39
imagePullPolicy: IfNotPresent
livenessProbe:
failureThreshold: 3
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-blockfrost-worker /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-blockfrost-worker
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-blockfrost-worker 2024-02-14 13:21:51.389566557 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-blockfrost-worker 2024-02-14 13:21:51.389566557 +0000
@@ -8,7 +8,7 @@
labels:
app: blockfrost-worker
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-blockfrost-worker
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-coingecko-proxy /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-coingecko-proxy
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-coingecko-proxy 2024-02-14 13:21:51.441566581 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-coingecko-proxy 2024-02-14 13:21:51.441566581 +0000
@@ -8,7 +8,7 @@
labels:
app: coingecko-proxy
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-coingecko-proxy
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-projector /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-projector
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-projector 2024-02-14 13:21:51.497566609 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-projector 2024-02-14 13:21:51.497566609 +0000
@@ -4,11 +4,11 @@
annotations:
deployment.kubernetes.io/revision: "8"
creationTimestamp: "2024-01-25T22:08:02Z"
- generation: 8
+ generation: 9
labels:
app: handle-projector
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-handle-projector
@@ -73,7 +73,7 @@
name: handle-owner-user.dev-preview-postgresql.credentials.postgresql.acid.zalan.do
- name: PROJECTION_NAMES
value: handle
- image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:2a9qvpsiryyfpp3y5q5zx918zb31gym2
+ image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:ckifi60diw14lzqh5d282y0i2fc79l39
imagePullPolicy: IfNotPresent
livenessProbe:
failureThreshold: 3
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-provider /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-provider
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-provider 2024-02-14 13:21:51.557566637 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-handle-provider 2024-02-14 13:21:51.557566637 +0000
@@ -4,11 +4,11 @@
annotations:
deployment.kubernetes.io/revision: "8"
creationTimestamp: "2024-01-25T22:08:02Z"
- generation: 8
+ generation: 9
labels:
app: handle-provider
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-handle-provider
@@ -81,7 +81,7 @@
value: handle
- name: USE_KORA_LABS
value: "true"
- image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:2a9qvpsiryyfpp3y5q5zx918zb31gym2
+ image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:ckifi60diw14lzqh5d282y0i2fc79l39
imagePullPolicy: IfNotPresent
livenessProbe:
failureThreshold: 3
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-pg-boss-worker /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-pg-boss-worker
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-pg-boss-worker 2024-02-14 13:21:51.609566662 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-pg-boss-worker 2024-02-14 13:21:51.609566662 +0000
@@ -4,11 +4,11 @@
annotations:
deployment.kubernetes.io/revision: "8"
creationTimestamp: "2024-01-25T22:08:02Z"
- generation: 8
+ generation: 9
labels:
app: pg-boss-worker
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-pg-boss-worker
@@ -97,7 +97,7 @@
value: https://smash.cardano-mainnet.iohk.io/api/v1
- name: STAKE_POOL_PROVIDER_URL
value: http://dev-preview-cardanojs-backend.dev-preview.svc.cluster.local
- image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:2a9qvpsiryyfpp3y5q5zx918zb31gym2
+ image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:ckifi60diw14lzqh5d282y0i2fc79l39
imagePullPolicy: IfNotPresent
livenessProbe:
failureThreshold: 3
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-projector /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-projector
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-projector 2024-02-14 13:21:51.665566687 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-projector 2024-02-14 13:21:51.665566687 +0000
@@ -4,11 +4,11 @@
annotations:
deployment.kubernetes.io/revision: "8"
creationTimestamp: "2024-01-25T22:08:02Z"
- generation: 8
+ generation: 9
labels:
app: stake-pool-projector
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-stake-pool-projector
@@ -73,7 +73,7 @@
name: stakepool-owner-user.dev-preview-postgresql.credentials.postgresql.acid.zalan.do
- name: PROJECTION_NAMES
value: stake-pool,stake-pool-metadata-job,stake-pool-metrics-job,stake-pool-rewards-job
- image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:2a9qvpsiryyfpp3y5q5zx918zb31gym2
+ image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:ckifi60diw14lzqh5d282y0i2fc79l39
imagePullPolicy: IfNotPresent
livenessProbe:
failureThreshold: 3
diff -u -N /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-provider /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-provider
--- /tmp/LIVE-747982017/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-provider 2024-02-14 13:21:51.721566716 +0000
+++ /tmp/MERGED-3834919188/apps.v1.Deployment.dev-preview.dev-preview-cardanojs-stake-pool-provider 2024-02-14 13:21:51.721566716 +0000
@@ -4,11 +4,11 @@
annotations:
deployment.kubernetes.io/revision: "8"
creationTimestamp: "2024-01-25T22:08:02Z"
- generation: 8
+ generation: 9
labels:
app: stake-pool-provider
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-stake-pool-provider
@@ -83,7 +83,7 @@
value: http://dev-preview-cardano-stack-metadata.dev-preview.svc.cluster.local
- name: USE_TYPEORM_STAKE_POOL_PROVIDER
value: "true"
- image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:2a9qvpsiryyfpp3y5q5zx918zb31gym2
+ image: 926093910549.dkr.ecr.us-east-1.amazonaws.com/cardano-services:ckifi60diw14lzqh5d282y0i2fc79l39
imagePullPolicy: IfNotPresent
livenessProbe:
failureThreshold: 3
diff -u -N /tmp/LIVE-747982017/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-backend-monitor /tmp/MERGED-3834919188/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-backend-monitor
--- /tmp/LIVE-747982017/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-backend-monitor 2024-02-14 13:21:51.781566744 +0000
+++ /tmp/MERGED-3834919188/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-backend-monitor 2024-02-14 13:21:51.781566744 +0000
@@ -5,7 +5,7 @@
generation: 1
labels:
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
instance: primary
name: lace-backend-monitor
namespace: dev-preview
diff -u -N /tmp/LIVE-747982017/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-handle-provider-monitor /tmp/MERGED-3834919188/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-handle-provider-monitor
--- /tmp/LIVE-747982017/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-handle-provider-monitor 2024-02-14 13:21:51.829566766 +0000
+++ /tmp/MERGED-3834919188/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-handle-provider-monitor 2024-02-14 13:21:51.829566766 +0000
@@ -5,7 +5,7 @@
generation: 1
labels:
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
instance: primary
name: lace-handle-provider-monitor
namespace: dev-preview
diff -u -N /tmp/LIVE-747982017/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-stake-pool-provider-monitor /tmp/MERGED-3834919188/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-stake-pool-provider-monitor
--- /tmp/LIVE-747982017/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-stake-pool-provider-monitor 2024-02-14 13:21:51.885566792 +0000
+++ /tmp/MERGED-3834919188/monitoring.coreos.com.v1.ServiceMonitor.dev-preview.lace-stake-pool-provider-monitor 2024-02-14 13:21:51.885566792 +0000
@@ -5,7 +5,7 @@
generation: 1
labels:
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
instance: primary
name: lace-stake-pool-provider-monitor
namespace: dev-preview
diff -u -N /tmp/LIVE-747982017/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-backend /tmp/MERGED-3834919188/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-backend
--- /tmp/LIVE-747982017/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-backend 2024-02-14 13:21:51.941566820 +0000
+++ /tmp/MERGED-3834919188/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-backend 2024-02-14 13:21:51.941566820 +0000
@@ -20,7 +20,7 @@
labels:
app: backend
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-backend
diff -u -N /tmp/LIVE-747982017/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-coingecko-proxy /tmp/MERGED-3834919188/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-coingecko-proxy
--- /tmp/LIVE-747982017/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-coingecko-proxy 2024-02-14 13:21:51.997566846 +0000
+++ /tmp/MERGED-3834919188/networking.k8s.io.v1.Ingress.dev-preview.dev-preview-cardanojs-coingecko-proxy 2024-02-14 13:21:51.997566846 +0000
@@ -18,7 +18,7 @@
labels:
app: coingecko-proxy
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-coingecko-proxy
diff -u -N /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-backend /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-backend
--- /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-backend 2024-02-14 13:21:51.117566467 +0000
+++ /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-backend 2024-02-14 13:21:51.117566467 +0000
@@ -5,7 +5,7 @@
labels:
app: backend
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-backend
diff -u -N /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-coingecko-proxy /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-coingecko-proxy
--- /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-coingecko-proxy 2024-02-14 13:21:51.169566478 +0000
+++ /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-coingecko-proxy 2024-02-14 13:21:51.169566478 +0000
@@ -5,7 +5,7 @@
labels:
app: coingecko-proxy
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-coingecko-proxy
diff -u -N /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-handle-provider /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-handle-provider
--- /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-handle-provider 2024-02-14 13:21:51.217566488 +0000
+++ /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-handle-provider 2024-02-14 13:21:51.221566489 +0000
@@ -5,7 +5,7 @@
labels:
app: handle-provider
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-handle-provider
diff -u -N /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-stake-pool-provider /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-stake-pool-provider
--- /tmp/LIVE-747982017/v1.Service.dev-preview.dev-preview-cardanojs-stake-pool-provider 2024-02-14 13:21:51.273566501 +0000
+++ /tmp/MERGED-3834919188/v1.Service.dev-preview.dev-preview-cardanojs-stake-pool-provider 2024-02-14 13:21:51.273566501 +0000
@@ -5,7 +5,7 @@
labels:
app: stake-pool-provider
app.kubernetes.io/managed-by: std-kubectl
- app.kubernetes.io/version: 638dddd96c0687f062913232785b9ecab19d69df
+ app.kubernetes.io/version: ecfef1e2837eba31f81fca4091b91d4aad7d5161
network: preview
release: dev-preview-cardanojs
name: dev-preview-cardanojs-stake-pool-provider |
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.
Nice work! 🚀
bda86db
to
afa91dc
Compare
@@ -12,7 +13,7 @@ const comparePolicyIdCanonically = (a: Trezor.CardanoAssetGroup, b: Trezor.Carda | |||
// PolicyId is always of the same length | |||
a.policyId > b.policyId ? 1 : -1; | |||
|
|||
const tokenMapToAssetGroup = (tokenMap: Cardano.TokenMap, isMint: boolean): Trezor.CardanoAssetGroup[] => { | |||
const tokenMapToAssetGroup: Transform<Cardano.TokenMap, Trezor.CardanoAssetGroup[]> = (tokenMap, isMint) => { |
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.
Interesting, so this just needed the generic type applied
@mkazlauskas approval was dismissed because of the last e69f4b6. Could you check it please ? |
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.
Well done @tomislavhoracek 🎉
Please squash |
e69f4b6
to
76ab44c
Compare
@rhyslbw squashed |
This PR introduces a TrezorKeyAgent Transform interface refactor
Context
as per Slack: https://input-output-rnd.slack.com/archives/C05GY0FSW73/p1707252256148049