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

LogBroadcaster feature flag #14354

Merged
merged 9 commits into from
Sep 9, 2024

Conversation

huangzhen1997
Copy link
Contributor

@huangzhen1997 huangzhen1997 commented Sep 5, 2024

Description

Create a feature flag to replace LogBroadcaster with the NullBroadcaster. When FeatureFlag is enabled (by default) we use the proper implementation of the LogBroadcaster. When FeatureFlag is disabled, we use Noop implementation.

Tickets:

BCFR-884

@huangzhen1997 huangzhen1997 marked this pull request as ready for review September 6, 2024 15:22
@huangzhen1997 huangzhen1997 requested review from a team as code owners September 6, 2024 15:22
@huangzhen1997 huangzhen1997 requested review from cedric-cordenier and removed request for a team September 6, 2024 15:22
Copy link
Collaborator

@dhaidashenko dhaidashenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left minor comment. Otherwise looks good to me

.changeset/kind-numbers-melt.md Outdated Show resolved Hide resolved
@patrick-dowell patrick-dowell added this pull request to the merge queue Sep 9, 2024
Merged via the queue into develop with commit bf6618d Sep 9, 2024
135 of 136 checks passed
@patrick-dowell patrick-dowell deleted the BCFR-884/adding-logbroadcaster-flag branch September 9, 2024 17:25
matYang pushed a commit to smartcontractkit/ccip that referenced this pull request Nov 14, 2024
Optional WS URL for RPCs to unblock new chain integration.
Charry-picks from:
smartcontractkit/chainlink#14354
smartcontractkit/chainlink#14373
smartcontractkit/chainlink#14534
smartcontractkit/chainlink#14364
smartcontractkit/chainlink#14929

---------

Co-authored-by: Joe Huang <joe.huang@smartcontract.com>
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.

4 participants