Skip to content

Commit

Permalink
Add temporary fix for name collision
Browse files Browse the repository at this point in the history
* This is fixed in another branch that must be merged
  • Loading branch information
JoshuaTheMiller committed Feb 8, 2023
1 parent a14dd4e commit a08dc89
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
5 changes: 5 additions & 0 deletions source/Gttsb.Core/GroupSyncer.cs
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,11 @@ private async Task<GroupSyncResult> SyncronizeGroupsAsync(string gitHubOrg, IEnu

foreach (var team in teamsControlledBySyncer)
{
if(team.Name == "NA")
{
continue;
}

if(!allTeams.TryGetValue(team.Name, out var specificTeam))
{
Console.WriteLine($"Creating team {team.Name}");
Expand Down
2 changes: 1 addition & 1 deletion source/Gttsb.Gh/InstalledGitHubFacade.cs
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ public async Task<IReadOnlyDictionary<string, GitHubTeam>> GetAllTeamsAsync(stri

var result = await graphQlClient.Run(query);

return result.ToDictionary(t=> t.Name, t => new GitHubTeam(t.DatabaseId, t.Name, t.Members.Select(m => new GitHubUser(m.Email, new ValidGitHubId(m.Login))).ToList()));
return result.ToDictionary(t=> t.Name, t => new GitHubTeam(t.DatabaseId, t.Name, t.Members.Select(m => new GitHubUser(m.Email, new ValidGitHubId(m.Login))).ToList()), StringComparer.CurrentCultureIgnoreCase);
}

public async Task AddTeamMemberAsync(GitHubTeam team, ValidGitHubId userGitHubId)
Expand Down

0 comments on commit a08dc89

Please sign in to comment.