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

Borrow proxy with PodOperation behavior #1854

Merged
merged 3 commits into from
Jun 3, 2024
Merged

Conversation

lemunozm
Copy link
Contributor

@lemunozm lemunozm commented Jun 3, 2024

Description

We want the ProxyType::Borrow to also behaves as the ProxyType::PodOperation. Thread: https://kflabs.slack.com/archives/C04GJQAM9P0/p1717154483749839

@lemunozm lemunozm added Q1-easy Can be done by primarily duplicating and adapting code. P5-soon Issue should be addressed soon. I8-enhancement An additional feature. labels Jun 3, 2024
@lemunozm lemunozm self-assigned this Jun 3, 2024
@lemunozm lemunozm added this to the Centrifuge 1029 milestone Jun 3, 2024
@lemunozm lemunozm changed the title PodOperation proxy with borrow behavior Borrow proxy with PodOperation behavior Jun 3, 2024
wischli
wischli previously approved these changes Jun 3, 2024
@@ -693,7 +693,8 @@ impl InstanceFilter<RuntimeCall> for ProxyType {
// Borrowers should be able to swap back and forth between local currencies and their variants
RuntimeCall::TokenMux(pallet_token_mux::Call::burn {..}) |
RuntimeCall::TokenMux(pallet_token_mux::Call::deposit {..}) |
RuntimeCall::TokenMux(pallet_token_mux::Call::match_swap {..})
RuntimeCall::TokenMux(pallet_token_mux::Call::match_swap {..}) |
ProxyType::PodOperation.filter(c)
Copy link
Contributor

Choose a reason for hiding this comment

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

I love the simplicity of this solution!

Copy link

codecov bot commented Jun 3, 2024

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 46.50%. Comparing base (2aac41b) to head (18c3ece).

Files Patch % Lines
runtime/altair/src/lib.rs 0.00% 3 Missing ⚠️
runtime/centrifuge/src/lib.rs 0.00% 3 Missing ⚠️
runtime/development/src/lib.rs 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1854      +/-   ##
==========================================
- Coverage   46.57%   46.50%   -0.08%     
==========================================
  Files         167      167              
  Lines       13098    13101       +3     
==========================================
- Hits         6100     6092       -8     
- Misses       6998     7009      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lemunozm lemunozm enabled auto-merge (squash) June 3, 2024 13:57
@lemunozm lemunozm requested a review from wischli June 3, 2024 13:58
@lemunozm lemunozm merged commit c6dea28 into main Jun 3, 2024
11 of 12 checks passed
@wischli wischli deleted the feat/pod-operation-proxy branch June 3, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I8-enhancement An additional feature. P5-soon Issue should be addressed soon. Q1-easy Can be done by primarily duplicating and adapting code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants