-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
feat(tempo): multisig #5029
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
feat(tempo): multisig #5029
Changes from 7 commits
2fbd761
3a70d72
a419dec
a320d6a
e2cc4b7
8afcd79
a485f97
f547f1f
7f04b9d
9417015
dc6aca7
c9fd81a
fe04735
6bdd0ad
b629125
ea53c8d
7d073d5
631dfed
a937eab
62a09c9
4c2fc3c
c297edf
0481f87
a581fa0
4565fc0
e315905
17140d7
f568fae
9a6fa51
6c30e37
2a41cbb
876a1e8
aff1bb0
a6cb3c5
533c718
015c98f
aa72fa5
2e459f8
ebe79a7
5397932
5ede31e
c299e0a
31c2ebb
9d00058
a46a11d
12e4cf3
07cb272
8cf30c2
08344ae
315dbcb
5bb99ae
9e249ca
5eeae1b
cbb006f
a98427c
cf62bf7
e172491
fadfa58
dc78be2
1626e1c
e42ac3f
01c9a8e
d1a23e4
6005360
054193c
ac00d5d
3b61969
7b65ac8
a250139
339187f
5691d0c
e7cd17b
a22eeba
202a332
e2f8c9e
9f1fda6
7292473
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| --- | ||
| "viem": patch | ||
| --- | ||
|
|
||
| `viem/tempo`: Added persistent multisig operation coordination and explicit owner approval actions. | ||
|
|
||
| ```ts | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The changeset continues after its past-tense summary with a fenced usage example, even though repository changeset entries must consist of a single sentence. Remove the example and retain only the summary sentence. AGENTS.md reference: AGENTS.md:L224-L227 Useful? React with 👍 / 👎. |
||
| import { Multisig, createClient, http } from 'viem/tempo' | ||
|
|
||
| const client = createClient({ | ||
| multisig: { store: Multisig.Store.memory() }, | ||
| transport: http(), | ||
| }) | ||
|
|
||
| const operation = await client.multisig.approveTransaction({ | ||
| ...request, | ||
| account: owner, | ||
| }) | ||
| ``` | ||
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
storageoption in this patchThis changeset declares a patch release even though the commit removes
storagefrom bothsignAuthorizationTokenand the Zonehttptransport. Existing JavaScript callers that prepopulate a custom store and passhttp(..., { storage })will have that option silently ignored, so the authorization token is read from the empty default store and authenticated requests fail; retainstorageas a deprecated alias or ship and document the rename as a major migration.AGENTS.md reference: AGENTS.md:L224-L227
Useful? React with 👍 / 👎.