[lib] introduce DISABLE_CONNECT_FARCASTER_ALERT gate
ClosedPublic
Actions

Authored by ginsu on Jun 18 2024, 11:35 PM.

Details

Reviewers

varun
ashoat

Commits

rCOMM0d9a8a7d2f8a: [lib] introduce DISABLE_CONNECT_FARCASTER_ALERT gate

Summary

Since we plan on flipping usingCommServicesAccessToken to true for devs we need to create a new gate for the connect farcaster prompt

Test Plan

flow + confirmed that the gate works as expected

Diff Detail

Repository

rCOMM Comm

Branch

eng-7462 (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Jun 18 2024, 11:35 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 18 2024, 11:35 PM

ginsu added a reviewer: varun.Jun 18 2024, 11:37 PM

Harbormaster completed remote builds in B29768: Diff 41478.Jun 18 2024, 11:52 PM

ginsu requested review of this revision.Jun 18 2024, 11:52 PM

ashoat accepted this revision.Jun 19 2024, 5:26 AM

This revision is now accepted and ready to land.Jun 19 2024, 5:26 AM

rebase before landing

This revision was landed with ongoing or failed builds.Jun 19 2024, 1:19 PM

Closed by commit rCOMM0d9a8a7d2f8a: [lib] introduce DISABLE_CONNECT_FARCASTER_ALERT gate (authored by ginsu). · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM0d9a8a7d2f8a: [lib] introduce DISABLE_CONNECT_FARCASTER_ALERT gate.

Harbormaster completed remote builds in B29771: Diff 41493.Jun 19 2024, 1:29 PM

Revision Contents
Changeset List

Path

Size

lib/

utils/

farcaster-utils.js

9 lines

push-alerts.js

4 lines

Diff 41478

View Options

lib/utils/farcaster-utils.js