-
Notifications
You must be signed in to change notification settings - Fork 13
Open
Labels
500 RSN Bountycontribution friendlyContributions outside Rosen Team are welcomedContributions outside Rosen Team are welcomed
Description
Description
Currently there is an environmental variable for the lock address of each chain supported by Rosen. There is no need for these env vars, because the addresses files from contract repo releases contain not only the lock address, but all other addresses that may be needed in the future.
All of the lock address env vars (which are in the form ofNEXT_PUBLIC_*_LOCK_ADDRESS) should be removed. A new NextJS server action should be introduced (e.g. getNetworkLockAddress), which accepts a network name and returns the lock address for that network (by reading address file contents, etc.). This action can be used in useTransaction hook. Note that there won't be any need for the useLockAddress hook anymore, and it should be removed.
Acceptance conditions
- All usages of lock address env vars should be updated based on the provided details (note that intermediary codes probably should be removed, too, such as the
lockAddressfield in Network objects) - A server action should be added based on the provided details
-
useLockAddresshook should be removed -
.env.exampleshould be updated
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
500 RSN Bountycontribution friendlyContributions outside Rosen Team are welcomedContributions outside Rosen Team are welcomed