sync preferred name;

remove settings.usernames
status-im · Sep 13, 2023 · 2b6f320 · 2b6f320
1 parent 7a9845c
commit 2b6f320
Show file tree

Hide file tree

Showing 23 changed files with 396 additions and 440 deletions.
diff --git a/api/geth_backend.go b/api/geth_backend.go
@@ -3,6 +3,7 @@ package api
 import (
 	"context"
 	"database/sql"
+	"encoding/json"
 	"errors"
 	"fmt"
 	"math/big"
@@ -12,6 +13,8 @@ import (
 	"sync"
 	"time"
 
+	"github.com/status-im/status-go/services/ens"
+
 	"github.com/imdario/mergo"
 
 	"github.com/ethereum/go-ethereum/common"
@@ -763,6 +766,12 @@ func (b *GethStatusBackend) GetSettings() (*settings.Settings, error) {
 	return &settings, nil
 }
 
+func (b *GethStatusBackend) GetEnsUsernames() ([]*ens.UsernameDetail, error) {
+	db := ens.NewEnsDatabase(b.appDB)
+	removed := false
+	return db.GetEnsUsernames(&removed)
+}
+
 func (b *GethStatusBackend) MigrateKeyStoreDir(acc multiaccounts.Account, password, oldDir, newDir string) error {
 	err := b.ensureDBsOpened(acc, password)
 	if err != nil {
@@ -812,7 +821,7 @@ func (b *GethStatusBackend) StartNodeWithAccount(acc multiaccounts.Account, pass
 
 func (b *GethStatusBackend) LoggedIn(keyUID string, err error) error {
 	if err != nil {
-		signal.SendLoggedIn(nil, nil, err)
+		signal.SendLoggedIn(nil, nil, nil, err)
 		return err
 	}
 	settings, err := b.GetSettings()
@@ -824,7 +833,18 @@ func (b *GethStatusBackend) LoggedIn(keyUID string, err error) error {
 		return err
 	}
 
-	signal.SendLoggedIn(account, settings, nil)
+	ensUsernames, err := b.GetEnsUsernames()
+	if err != nil {
+		return err
+	}
+	var ensUsernamesJSON json.RawMessage
+	if ensUsernames != nil {
+		ensUsernamesJSON, err = json.Marshal(ensUsernames)
+		if err != nil {
+			return err
+		}
+	}
+	signal.SendLoggedIn(account, settings, ensUsernamesJSON, nil)
 	return nil
 }
 

diff --git a/appdatabase/migrations/bindata.go b/appdatabase/migrations/bindata.go
diff --git a/appdatabase/migrationsprevnodecfg/bindata.go b/appdatabase/migrationsprevnodecfg/bindata.go
diff --git a/mailserver/migrations/bindata.go b/mailserver/migrations/bindata.go
diff --git a/multiaccounts/migrations/bindata.go b/multiaccounts/migrations/bindata.go
diff --git a/multiaccounts/settings/columns.go b/multiaccounts/settings/columns.go
@@ -255,9 +255,7 @@ var (
 	PreferredName = SettingField{
 		reactFieldName: "preferred-name",
 		dBColumnName:   "preferred_name",
-		// TODO resolve issue 9 https://github.com/status-im/status-mobile/pull/13053#issuecomment-1075336559
 		syncProtobufFactory: &SyncProtobufFactory{
-			inactive:          true, // Remove after issue is resolved
 			fromInterface:     preferredNameProtobufFactory,
 			fromStruct:        preferredNameProtobufFactoryStruct,
 			valueFromProtobuf: StringFromSyncProtobuf,
@@ -399,17 +397,6 @@ var (
 		dBColumnName:   "use_mailservers",
 		valueHandler:   BoolHandler,
 	}
-	Usernames = SettingField{
-		reactFieldName: "usernames",
-		dBColumnName:   "usernames",
-		valueHandler:   JSONBlobHandler,
-		syncProtobufFactory: &SyncProtobufFactory{
-			fromInterface:     usernamesProtobufFactory,
-			fromStruct:        usernamesProtobufFactoryStruct,
-			valueFromProtobuf: BytesFromSyncProtobuf,
-			protobufType:      protobuf.SyncSetting_ENS_USERNAMES,
-		},
-	}
 	WakuBloomFilterMode = SettingField{
 		reactFieldName: "waku-bloom-filter-mode",
 		dBColumnName:   "waku_bloom_filter_mode",
@@ -519,7 +506,6 @@ var (
 		TelemetryServerURL,
 		TestNetworksEnabled,
 		UseMailservers,
-		Usernames,
 		WakuBloomFilterMode,
 		WalletRootAddress,
 		WalletSetUpPassed,

diff --git a/multiaccounts/settings/database.go b/multiaccounts/settings/database.go
@@ -614,11 +614,6 @@ func (db *Database) GetInstalledStickerPacks() (rst *json.RawMessage, err error)
 	return
 }
 
-func (db *Database) Usernames() (rst *json.RawMessage, err error) {
-	err = db.makeSelectRow(Usernames).Scan(&rst)
-	return
-}
-
 func (db *Database) GetPendingStickerPacks() (rst *json.RawMessage, err error) {
 	err = db.makeSelectRow(StickersPacksPending).Scan(&rst)
 	return

diff --git a/multiaccounts/settings/sync_protobuf_factories.go b/multiaccounts/settings/sync_protobuf_factories.go
@@ -515,32 +515,6 @@ func mnemonicRemovedProtobufFactoryStruct(s Settings, clock uint64, chatID strin
 	return buildRawMnemonicRemovedSyncMessage(s.MnemonicRemoved, clock, chatID)
 }
 
-// Usernames
-
-func buildRawUsernamesSyncMessage(v []byte, clock uint64, chatID string) (*common.RawMessage, *protobuf.SyncSetting, error) {
-	pb := &protobuf.SyncSetting{
-		Type:  protobuf.SyncSetting_ENS_USERNAMES,
-		Value: &protobuf.SyncSetting_ValueBytes{ValueBytes: v},
-		Clock: clock,
-	}
-	rm, err := buildRawSyncSettingMessage(pb, chatID)
-	return rm, pb, err
-}
-
-func usernamesProtobufFactory(value interface{}, clock uint64, chatID string) (*common.RawMessage, *protobuf.SyncSetting, error) {
-	v, err := assertBytes(value)
-	if err != nil {
-		return nil, nil, err
-	}
-
-	return buildRawUsernamesSyncMessage(v, clock, chatID)
-}
-
-func usernamesProtobufFactoryStruct(s Settings, clock uint64, chatID string) (*common.RawMessage, *protobuf.SyncSetting, error) {
-	us := extractJSONRawMessage(s.Usernames)
-	return buildRawUsernamesSyncMessage(us, clock, chatID)
-}
-
 // IncludeWatchOnlyAccount
 
 func buildRawIncludeWatchOnlyAccountSyncMessage(v bool, clock uint64, chatID string) (*common.RawMessage, *protobuf.SyncSetting, error) {

diff --git a/protocol/anonmetrics/migrations/migrations.go b/protocol/anonmetrics/migrations/migrations.go