implement feedback

Cargo.toml

@@ -59,14 +59,14 @@ sqlx = {version = "0", features = [
 ]}
 
 helium-crypto = {version = "0.6.8", features=["sqlx-postgres", "multisig"]}
-helium-proto = {git = "https://github.com/helium/proto", branch = "jg/allow-pcs-mobile-keys", features = ["services"]}
+helium-proto = {git = "https://github.com/helium/proto", branch = "master", features = ["services"]}
 hextree = "*"
 solana-client = "1.14"
 solana-sdk = "1.14"
 solana-program = "1.11"
 spl-token = "3.5.0"
 reqwest = {version = "0", default-features=false, features = ["gzip", "json", "rustls-tls"]}
-beacon = { git = "https://github.com/helium/proto", branch = "jg/allow-pcs-mobile-keys" }
+beacon = { git = "https://github.com/helium/proto", branch = "master" }
 humantime = "2"
 metrics = "0"
 metrics-exporter-prometheus = "0"

mobile_config/migrations/20230708171204_add_pcs_key_role.sql

@@ -1 +1 @@
-ALTER TYPE key_role ADD VALUE IF NOT EXISTS 'pcs';-- Add migration script here
+ALTER TYPE key_role ADD VALUE IF NOT EXISTS 'pcs';

mobile_config/src/lib.rs

@@ -80,11 +80,11 @@ impl From<ProtoKeyRole> for KeyRole {
 impl std::fmt::Display for KeyRole {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         let s = match self {
-            Self::Administrator => "Administrator",
-            Self::Carrier => "Carrier",
-            Self::Oracle => "Oracle",
-            Self::Router => "Router",
-            Self::Pcs => "PCS",
+            Self::Administrator => "administrator",
+            Self::Carrier => "carrier",
+            Self::Oracle => "oracle",
+            Self::Router => "router",
+            Self::Pcs => "pcs",
         };
         f.write_str(s)
     }

mobile_config_cli/src/lib.rs

@@ -102,9 +102,9 @@ impl From<NetworkKeyRole> for i32 {
 impl Display for NetworkKeyRole {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {
-            NetworkKeyRole::MobileCarrier => write!(f, "Carrier"),
-            NetworkKeyRole::MobileRouter => write!(f, "Router"),
-            NetworkKeyRole::MobilePcs => write!(f, "PCS"),
+            NetworkKeyRole::MobileCarrier => write!(f, "carrier"),
+            NetworkKeyRole::MobileRouter => write!(f, "router"),
+            NetworkKeyRole::MobilePcs => write!(f, "pcs"),
         }
     }
 }