Skip to content

Commit

Permalink
fix: fix up creator role for latest daily server
Browse files Browse the repository at this point in the history
  • Loading branch information
revam committed Dec 31, 2024
1 parent 3ef0aec commit e9f4f76
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Shokofin/API/Info/EpisodeInfo.cs
Original file line number Diff line number Diff line change
Expand Up @@ -380,7 +380,7 @@ public async Task<EpisodeImages> GetImages(CancellationToken cancellationToken)
{ roleProvider, roles[0].Staff.Id!.Value.ToString() },
},
},
CreatorRoleType.Seiyuu => new PersonInfo {
CreatorRoleType.Actor => new PersonInfo {
Type = PersonKind.Actor,
Name = roles[0].Staff.Name,
Role = roles.Select(role => role?.Character?.Name ?? string.Empty).Where(role => !string.IsNullOrEmpty(role)).Order().Join(" / "),
Expand Down
9 changes: 7 additions & 2 deletions Shokofin/API/Models/Role.cs
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ public class Role : IEquatable<Role> {

/// <summary>
/// The character played, the <see cref="Role.Type"/> is of type
/// <see cref="CreatorRoleType.Seiyuu"/>.
/// <see cref="CreatorRoleType.Actor"/>.
/// </summary>
public Person? Character { get; set; }

Expand Down Expand Up @@ -101,7 +101,12 @@ public enum CreatorRoleType {
/// <summary>
/// Voice actor or voice actress.
/// </summary>
Seiyuu,
Actor,

/// <summary>
/// Synonym of <see cref="CreatorRoleType.Actor"/> for backwards compatibility for a few server versions.
/// </summary>
Seiyuu = Actor,

/// <summary>
/// This can be anything involved in writing the show.
Expand Down

0 comments on commit e9f4f76

Please sign in to comment.