-
Notifications
You must be signed in to change notification settings - Fork 66
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
script to modify the state history retention period via the safe sdk #2012
base: main
Are you sure you want to change the base?
script to modify the state history retention period via the safe sdk #2012
Conversation
…in-via-the-safe-sdk
…in-via-the-safe-sdk
* @param {string} value - The value to be sent with the transaction | ||
* @returns {Promise<any>} - A promise that resolves to the Safe transaction object | ||
*/ | ||
async function createSafeTransaction( |
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.
we could move this to utils.ts
as it can be used by other files in the future
); | ||
|
||
console.log( | ||
`The other owners of the Safe Multisig wallet need to sign the transaction via the Safe UI https://app.safe.global/transactions/queue?safe=sep:${safeAddress}`, |
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.
nit: Proposal created successfully. The other owners of the Safe Multisig wallet need to sign the transaction via the Safe UI https://app.safe.global/transactions/queue?safe=sep:${safeAddress}
* @param {number} retention_period - The state history retention period in seconds | ||
* @returns {string} - Encoded transaction data | ||
*/ | ||
function createStateHistoryRetentionPeriodTxData(retention_period: number): string { |
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.
nit: does this have to be a function? it is just two lines so we can prob merge this with proposeSetStateHistoryRetentionTransaction
): Promise<LocalSafeTransaction> { | ||
// Prepare the safe transaction data with the contract address, data, and value | ||
let safeTransactionData = createSafeTransactionData(contractAddress, data, value); | ||
console.log("data hex: ", data); |
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.
console.log("Safe Transaction Data hex: ", data)
Closes #2008
This PR: