Skip to content
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

Merged
merged 4 commits into from
Sep 27, 2023
Merged

Conversation

paullinator
Copy link
Member

@paullinator paullinator commented Sep 21, 2023

CHANGELOG

none

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Copy link
Collaborator

@Jon-edge Jon-edge left a 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))
Copy link
Collaborator

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

Copy link
Member Author

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

@paullinator
Copy link
Member Author

Do not merge. found a bug that will need a fixup

@paullinator paullinator force-pushed the paul/stableSavers branch 3 times, most recently from 491e07f to f30bef3 Compare September 26, 2023 00:55
@paullinator
Copy link
Member Author

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.

@paullinator paullinator reopened this Sep 26, 2023
@paullinator paullinator force-pushed the paul/stableSavers branch 2 times, most recently from 7ef8b2e to 64a1f90 Compare September 26, 2023 04:10
@paullinator paullinator changed the title Paul/stable savers Dynamic TC Savers pools. Was: Stable savers Sep 26, 2023
@@ -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 }))
Copy link
Collaborator

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

Copy link
Member Author

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.

@paullinator
Copy link
Member Author

/rebase

@paullinator
Copy link
Member Author

/rebase_autosquash

@paullinator paullinator merged commit b573eaf into develop Sep 27, 2023
2 checks passed
@paullinator paullinator deleted the paul/stableSavers branch September 27, 2023 01:05
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.

2 participants