Skip to content

[PM-1271] Removed StorageMediatorOptions to plain parameters#2397

Merged
fedemkr merged 2 commits intomasterfrom
PM-1271-storage-mediator-option-params
Mar 2, 2023
Merged

[PM-1271] Removed StorageMediatorOptions to plain parameters#2397
fedemkr merged 2 commits intomasterfrom
PM-1271-storage-mediator-option-params

Conversation

@fedemkr
Copy link
Member

@fedemkr fedemkr commented Mar 2, 2023

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Removed the StorageMediatorOptions and replaced it with plain parameters approach

Code changes

  • StorageMediatorService: Changed methods to use plain parameters approach instead of the object
  • StorageMediatorOptions: Removed

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@fedemkr fedemkr requested a review from a team March 2, 2023 19:19
@fedemkr fedemkr changed the title [PM-1271] Removed StorageMediatorOptions to a plain parameter [PM-1271] Removed StorageMediatorOptions to plain parameters Mar 2, 2023
@fedemkr fedemkr merged commit 3f86bb0 into master Mar 2, 2023
@fedemkr fedemkr deleted the PM-1271-storage-mediator-option-params branch March 2, 2023 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants