Skip to content

Commit

Permalink
feat: enable paid checkpoint for all
Browse files Browse the repository at this point in the history
  • Loading branch information
ekremney committed Jul 1, 2024
1 parent da8ab6d commit 080fbb4
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 7 deletions.
2 changes: 1 addition & 1 deletion modules/defaults.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,4 @@
* governing permissions and limitations under the License.
*/
export const KNOWN_PROPERTIES = ['weight', 'id', 'referer', 'checkpoint', 't', 'source', 'target', 'cwv', 'CLS', 'FID', 'LCP', 'INP', 'TTFB'];
export const DEFAULT_TRACKING_EVENTS = ['click', 'cwv', 'form', 'enterleave', 'viewblock', 'viewmedia', 'loadresource', 'utm'];
export const DEFAULT_TRACKING_EVENTS = ['click', 'cwv', 'form', 'enterleave', 'viewblock', 'viewmedia', 'loadresource', 'utm', 'paid'];
1 change: 0 additions & 1 deletion modules/fflags.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,5 @@ export const fflags = {
enabled: (flag, callback) => fflags.has(flag) && callback(),
disabled: (flag, callback) => !fflags.has(flag) && callback(),
onetrust: [543, 770, 1136],
ads: [1139, 543],
email: [1139, 543],
};
1 change: 0 additions & 1 deletion modules/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ function optedIn(checkpoint, data) {
function getCollectionConfig() {
// eslint-disable-next-line max-len
fflags.enabled('onetrust', () => DEFAULT_TRACKING_EVENTS.push('consent'));
fflags.enabled('ads', () => DEFAULT_TRACKING_EVENTS.push('paid'));
fflags.enabled('email', () => DEFAULT_TRACKING_EVENTS.push('email'));
return DEFAULT_TRACKING_EVENTS;
}
Expand Down
4 changes: 0 additions & 4 deletions test/unit/fflags.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,6 @@ describe('test fflags', () => {
expect(fflags.onetrust).to.be.an('array');
});

it('fflags.ads is an array', () => {
expect(fflags.ads).to.be.an('array');
});

it('fflags.email is an array', () => {
expect(fflags.email).to.be.an('array');
});
Expand Down

0 comments on commit 080fbb4

Please sign in to comment.