fix: code fixed to allow loading all the descriptors in the UI (#6648)

This commit is contained in:
sagargolu 2025-05-20 23:12:34 +05:30 committed by GitHub
parent 1c2326ed80
commit de8c692f48
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -24,7 +24,7 @@ public class ActivityRegistry(IActivityDescriber activityDescriber, IEnumerable<
}
/// <inheritdoc />
public IEnumerable<ActivityDescriptor> ListAll() => _activityDescriptors.Values.DistinctBy(x => x.TypeName);
public IEnumerable<ActivityDescriptor> ListAll() => _activityDescriptors.Values;
/// <inheritdoc />
public IEnumerable<ActivityDescriptor> ListByProvider(Type providerType) => _providedActivityDescriptors.TryGetValue(providerType, out var descriptors) ? descriptors : ArraySegment<ActivityDescriptor>.Empty;
@ -123,13 +123,13 @@ public class ActivityRegistry(IActivityDescriber activityDescriber, IEnumerable<
foreach (var modifier in modifiers)
modifier.Modify(descriptor);
// If the descriptor already exists, replace it. But log a warning.
// If the descriptor already exists, replace it. But log a warning.
if (activityDescriptors.TryGetValue((descriptor.TypeName, descriptor.Version), out var existingDescriptor))
{
// Remove the existing descriptor from the providerDescriptors collection.
// Remove the existing descriptor from the providerDescriptors collection.
providerDescriptors.Remove(existingDescriptor);
// Log a warning.
// Log a warning.
logger.LogWarning("Activity descriptor {ActivityType} v{ActivityVersion} was already registered. Replacing with new descriptor", descriptor.TypeName, descriptor.Version);
}