Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(web, api, worker): Remove multi-provider feature flag #4396

Closed
wants to merge 8 commits into from

Conversation

rifont
Copy link
Contributor

@rifont rifont commented Oct 4, 2023

What change does this PR introduce?

This PR removes the multi-provider feature flag from the Web, Worker, and API packages. It also adds documentation to support new users to get started with local API development.

Why was this change needed?

The multi-provider feature has been successfully implemented in production for an adequate length of time.

Other information (Screenshots)

The following redundant use-cases have been removed:

  • DeactivateSimilarChannelIntegrations
  • DisableNovuIntegration

@linear
Copy link

linear bot commented Oct 4, 2023

NV-2907 Remove the IS_MULTI_PROVIDER_CONFIGURATION_ENABLED feature flag

What?

We would like to remove the IS_MULTI_PROVIDER_CONFIGURATION_ENABLED feature flag and all associated "false" statement code.

Why? (Context)

The feature flag IS_MULTI_PROVIDER_CONFIGURATION_ENABLED is running on the prod for quite some time.

Definition of Done

  • the feature flag IS_MULTI_PROVIDER_CONFIGURATION_ENABLEDand all associated "false" statement code is removed

@rifont
Copy link
Contributor Author

rifont commented Oct 5, 2023

@rifont rifont closed this Oct 5, 2023
@rifont rifont deleted the NV-2907-remove-multiprovider-ff branch November 7, 2024 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant