What version of FlowSynx?
1.2.2
Describe the bug
In the Workflows class, the Map method currently defines the parameter as rateLimitPolicy, while the base class (EndpointGroupBase) uses rateLimitPolicyName.
To maintain consistency and avoid potential confusion or mismatched overrides, the parameter should be renamed to match the base class definition.
File: src/FlowSynx/Endpoints/Workflows.cs
Method: Map
Proposed change
public override void Map(WebApplication app, string rateLimitPolicyName)
Rationale
- Ensures consistency with the base class method signature.
- Improves code readability and prevents potential confusion when overriding methods.
- Aligns with standard naming conventions in the codebase.
What version of FlowSynx?
Describe the bug
In the
Workflowsclass, theMapmethod currently defines the parameter as rateLimitPolicy, while the base class (EndpointGroupBase) uses rateLimitPolicyName.To maintain consistency and avoid potential confusion or mismatched overrides, the parameter should be renamed to match the base class definition.
File:
src/FlowSynx/Endpoints/Workflows.csMethod:
MapProposed change
Rationale