-
Notifications
You must be signed in to change notification settings - Fork 258
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
Dynamic TC Savers pools. Was: Stable savers #4479
Conversation
df2e660
to
c83c259
Compare
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.
Approved with minor comment
}) | ||
policiesInitialized = true | ||
} catch (e) { | ||
console.log('Error initializing Thorchain Savers policies', String(e)) |
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.
Would this frequently happen? If not, seems useful to promote this to a console warn or error
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.warn is annoying in that it shows as a yellow box in the debugger. It's not critical as it can happen if there is a networking failure
Do not merge. found a bug that will need a fixup |
491e07f
to
f30bef3
Compare
This implementation will not work as I've been notified by TC devs that we need to use a smart contract for token deposits. This requires a major overhaul to make the code similar to the swap code that uses memos. Closing this task as a new one will be estimated and planned out. |
7ef8b2e
to
64a1f90
Compare
64a1f90
to
5fdf02a
Compare
5fdf02a
to
69809d6
Compare
@@ -197,7 +197,7 @@ function EditTokenSceneComponent(props: Props) { | |||
* In the future this scene may need to handle other weird networks | |||
* where the networkLocation has other contents. | |||
*/ | |||
const asMaybeContractLocation = asMaybe(asObject({ contractAddress: asString })) | |||
export const asMaybeContractLocation = asMaybe(asObject({ contractAddress: asString })) |
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.
Might be a good time to move /aave/cleaners/asEthTokenContractAddress.ts to a
common location and add asMaybeContractLocation to it
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.
Will do for the full Thorchain Savers with token task.
5bfe115
to
62c61cb
Compare
/rebase |
/rebase_autosquash |
62c61cb
to
6a03b01
Compare
CHANGELOG
noneDependencies
noneRequirements
If you have made any visual changes to the GUI. Make sure you have: