Skip to content

Allow tuples as parameters for smartcontract proposals#312

Open
fedepo wants to merge 4 commits intokalidao:mainfrom
fedepo:main
Open

Allow tuples as parameters for smartcontract proposals#312
fedepo wants to merge 4 commits intokalidao:mainfrom
fedepo:main

Conversation

@fedepo
Copy link
Copy Markdown

@fedepo fedepo commented Oct 13, 2023

Some smartcontracts require tuple[] as input parameters.
When that's the case, the user will want to pass a JSON formatted array (as they'd do on Etherscan).

This little fix checks if the input provided is a JSON. If it is, it will parse the JSON and pass it as an array to the function that creates the payload. If it's not a JSON string, it will just pass it as is.

for context, discord thread is here:
https://discord.com/channels/923399898769018921/1161730434120548413

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 13, 2023

@fedepo is attempting to deploy a commit to the KaliCo Team on Vercel.

A member of the Team first needs to authorize it.

@fedepo fedepo marked this pull request as ready for review October 13, 2023 06:20
@fedepo fedepo marked this pull request as draft October 13, 2023 06:20
@fedepo fedepo marked this pull request as ready for review October 13, 2023 06:21
@fedepo fedepo marked this pull request as draft October 13, 2023 06:36
@fedepo fedepo closed this Oct 13, 2023
@fedepo fedepo reopened this Oct 13, 2023
@fedepo fedepo marked this pull request as ready for review October 13, 2023 07:14
@fedepo
Copy link
Copy Markdown
Author

fedepo commented Apr 25, 2024

@audsssy any chance you can take a look at this / merge it? We tested this quite a bit and used on our verbal deployment successfully. thanks.

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.

1 participant