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

dapp: adapt update message to express the risk #3091

Conversation

weilbith
Copy link
Contributor

This is just a "hot fix". That means that the next update will include a breaking change in terms of new contracts. Users should not simply update without withdrawing their funds first. In future this will be replaced with a proper feature that is able to determine of which kind an update is to alternate the update message.

This is just a "hot fix". That means that the next update will include
a breaking change in terms of new contracts. Users should not simply
update without withdrawing their funds first.
In future this will be replaced with a proper feature that is able to
determine of which kind an update is to alternate the update message.
@weilbith
Copy link
Contributor Author

This change has been done together with @DinaBB

There is no need for a CHANGELOG entry here.

@weilbith weilbith added this to In progress in Light Client on Arbitrum via automation Mar 25, 2022
@weilbith
Copy link
Contributor Author

The above referenced proper feature can be tracked in #3090

Copy link
Contributor

@taleldayekh taleldayekh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 😃

@codecov

This comment was marked as resolved.

@weilbith weilbith merged commit 05a4e32 into raiden-network:master Mar 25, 2022
Light Client on Arbitrum automation moved this from In progress to Done Mar 25, 2022
@weilbith weilbith deleted the feature/dapp-adapt-update-message-to-express-risk branch March 25, 2022 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants