-
Notifications
You must be signed in to change notification settings - Fork 18
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
Turbo cellar permissions #224
Conversation
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.
Some comments and questions.
(CELLAR_RYBTC, ADAPTOR_LEGACY_CELLAR_V1), | ||
(CELLAR_RYBTC, ADAPTOR_COLLATERAL_F_TOKEN_V1), | ||
(CELLAR_RYBTC, ADAPTOR_DEBT_F_TOKEN_V1), | ||
pub const ALLOWED_V2_2_CATALOGUE_ADAPTORS: [(&str, &str); 0] = []; |
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.
Have we confirmed all the 2.2 adaptor add calls were made?
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.
No, looking for feedback from them in this PR
(CELLAR_RYBTC, 196), | ||
(CELLAR_RYBTC, 197), | ||
(CELLAR_RYBTC, 198), | ||
pub const ALLOWED_V2_2_CATALOGUE_POSITIONS: [(&str, u32); 0] = []; |
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.
Same question here re: confirming calls have already been made.
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.
LGTM
No description provided.