Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add deep freeze feature (XLS-77d) #5187
Add deep freeze feature (XLS-77d) #5187
Changes from 12 commits
e57d068
a0d48ae
4aac190
306a112
f5efa31
8e1977f
a162663
c7c26ac
805102a
de58a26
91d291a
e0d59ef
ffa6ccd
f9107a7
1f9ecb8
010f36d
6bae95d
b5a10ec
a1d7dac
c971db5
9f17d9f
d75894a
03f7b5d
ae5fae2
c60d491
0d8855a
70235b6
a73ea01
411a0f0
4c5c9cc
9f21f5d
8c40ab9
c5aa145
851e619
290b908
9db6765
7ba80fe
79a4da3
c1778e5
edf659e
d7360c9
536fff2
f33eba9
7fcd042
5d032e4
0436866
fe9346f
69e24bc
9aa72eb
4d58b4f
f4c3e1b
a891d62
bd0e5cc
024c16c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Don't need to repeat the code. Just add a conditional error:
And then use it in
pay
: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.
Actually, you shouldn't test here and other updated unit-tests if the feature is disabled because
trust
is going to fail. You need a separate test for disabled feature inSetTrust
and also add toSetTrust
tests for invalid combination of the flags. Also need to addOfferCreate
and offer crossing, if not added yet, toOffer
.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.
AMM Create, Deposit, Withdraw are not impacted in any way by deep freeze, correct? Just want to check.
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.
from design perspective it should not, deep freeze imposes a superset of restrictions of the existing regular freeze feature. so any restrictions in these transactions should have already been implemented through the freeze feature already.
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.
Should add deep freeze check to
CreateCheck
transactor.