Fix github mapping variant

oxidecomputer · Oct 16, 2023 · 68beaeb · 68beaeb
1 parent 2d71422
commit 68beaeb
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -381,7 +381,7 @@ cargo run -p rfd-cli mapper create --json-body /dev/stdin <<EOM
   "name": "add_github_user",
   "max_activations": 1,
   "rule": {
-    "rule": "git_hub_username",
+    "rule": "github_username",
     "github_username": "githubuser",
     "groups": [
       "friend-of-company"

diff --git a/rfd-api-spec.json b/rfd-api-spec.json
@@ -2669,7 +2669,7 @@
               "rule": {
                 "type": "string",
                 "enum": [
-                  "git_hub_username"
+                  "github_username"
                 ]
               }
             },

diff --git a/rfd-api/src/mapper/mod.rs b/rfd-api/src/mapper/mod.rs
@@ -64,6 +64,7 @@ impl TryFrom<Mapper> for Mapping {
 pub enum MappingRules {
     EmailAddress(EmailAddressMapper),
     EmailDomain(EmailDomainMapper),
+    #[serde(rename = "github_username")]
     GitHubUsername(GitHubUsernameMapper),
 }
 

diff --git a/rfd-sdk/src/generated/sdk.rs b/rfd-sdk/src/generated/sdk.rs
@@ -720,12 +720,12 @@ pub mod types {
             #[serde(default = "defaults::mapping_rules_email_domain_permissions")]
             permissions: PermissionsForApiPermission,
         },
-        #[serde(rename = "git_hub_username")]
-        GitHubUsername {
+        #[serde(rename = "github_username")]
+        GithubUsername {
             github_username: String,
             #[serde(default, skip_serializing_if = "Vec::is_empty")]
             groups: Vec<String>,
-            #[serde(default = "defaults::mapping_rules_git_hub_username_permissions")]
+            #[serde(default = "defaults::mapping_rules_github_username_permissions")]
             permissions: PermissionsForApiPermission,
         },
     }
@@ -4187,7 +4187,7 @@ pub mod types {
             super::PermissionsForApiPermission(vec![])
         }
 
-        pub(super) fn mapping_rules_git_hub_username_permissions(
+        pub(super) fn mapping_rules_github_username_permissions(
         ) -> super::PermissionsForApiPermission {
             super::PermissionsForApiPermission(vec![])
         }