Skip to content

Commit

Permalink
Lets store the node_id not the user_id since we can use it in graphql…
Browse files Browse the repository at this point in the history
… or the REST api
  • Loading branch information
coreyja committed Nov 26, 2023
1 parent 7f1200a commit c81358a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions server/src/http_server/auth/routes.rs
Original file line number Diff line number Diff line change
Expand Up @@ -250,10 +250,10 @@ impl GithubUser {
}
}

fn id(&self) -> i64 {
fn id(&self) -> &str {
match &self {
GithubUser::PrivateUser { id, .. } => *id,
GithubUser::PublicUser { id, .. } => *id,
GithubUser::PrivateUser { node_id, .. } => node_id,
GithubUser::PublicUser { node_id, .. } => node_id,
}
}
}

0 comments on commit c81358a

Please sign in to comment.