[donor-voice] patch cw migration (#423)

Co-authored-by: Rupert Adagio <[email protected]>
Co-authored-by: Franci LeLento <[email protected]>
Co-authored-by: Reginald Ritardando <[email protected]>

Author: 4 people

framework/libra-framework/sources/ol_sources/migrations/filo_migration.move

@@ -1,19 +1,29 @@
 module ol_framework::filo_migration {
   use ol_framework::activity;
+  use ol_framework::donor_voice;
   use ol_framework::founder;
   use ol_framework::page_rank_lazy;
   use ol_framework::slow_wallet;
+
   use ol_framework::vouch;
+  use std::signer;
+  use std::error;
 
   friend diem_framework::transaction_validation;
 
+  /// Cannot be used by donor voice accounts
+  const EDONOR_VOICE: u64 = 1;
+
   // Welcome to Level 8
 
   // It's a quest, it's a quest
   // For the golden prize, the prize that never dies
   // For a dream that's worth the fight, worth the chase
   // It's a journey, it's a race.
   public entry fun maybe_migrate(user_sig: &signer) {
+    // community wallets should not do this migration
+    assert!(!donor_voice::is_donor_voice(signer::address_of(user_sig)), error::invalid_argument(EDONOR_VOICE));
+
     // Rising up, back on the street
     // Did my time, took my chances
     // Went the distance, now I'm back on my feet

framework/libra-framework/sources/ol_sources/slow_wallet.move

@@ -12,6 +12,7 @@ module ol_framework::slow_wallet {
   use std::vector;
   use diem_framework::account;
   use diem_framework::system_addresses;
+  use ol_framework::community_wallet;
   use ol_framework::libra_coin;
   use ol_framework::reauthorization;
   use ol_framework::sacred_cows;
@@ -305,11 +306,18 @@ module ol_framework::slow_wallet {
         return 0
       };
 
-      if (exists<SlowWallet>(addr)) {
+      if (
+        exists<SlowWallet>(addr) &&
+        // patches bug in v8 where CW may have initialized as user account
+        !community_wallet::is_init(addr)
+      ) {
         let s = borrow_global<SlowWallet>(addr);
         return s.unlocked
       };
 
+      // NOTE: we would use community_wallet_advance::total_credit_available
+      // for community wallets but it causes a dependency loop
+
       // this is a normal account, so return the normal balance
 
       libra_coin::balance(addr)