From 2727b74f80b3eacf432bdd2fbc5063c9bda247a4 Mon Sep 17 00:00:00 2001
From: Catherine Smith <eastandwestwind@gmail.com>
Date: Sat, 11 Nov 2023 02:53:48 +0100
Subject: [PATCH] Prod 1328 Track more granular consent method (#4419)

---
 CHANGELOG.md                                  |   1 +
 .../src/types/api/models/ConsentMethod.ts     |   6 +-
 .../src/components/ConsentButtons.tsx         |  17 ++-
 .../src/components/notices/NoticeOverlay.tsx  |  21 +++-
 .../src/components/tcf/TcfConsentButtons.tsx  |   8 +-
 .../src/components/tcf/TcfOverlay.tsx         |  24 ++--
 clients/fides-js/src/lib/consent-types.ts     |   8 +-
 clients/fides-js/src/lib/events.ts            |   6 +-
 clients/fides-js/src/lib/preferences.ts       |   3 +-
 clients/fides-js/src/services/api.ts          |   3 +
 .../cypress/e2e/consent-banner-tcf.cy.ts      | 103 ++++++++++++------
 .../cypress/e2e/consent-banner.cy.ts          |  37 ++++++-
 src/fides/api/models/privacy_preference.py    |   6 +-
 13 files changed, 174 insertions(+), 69 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 38d4319c88..a810979d87 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,6 +19,7 @@ The types of changes are:
 
 ### Added
 - Adds support for custom get experiences fn and custom patch notices served fn [#4410](https://github.com/ethyca/fides/pull/4410)
+- Adds more granularity to tracking consent method, updates custom savePreferencesFn and FidesUpdated event to take consent method [#4419](https://github.com/ethyca/fides/pull/4419)
 
 ### Changed
 - Button ordering in fides.js UI [#4407](https://github.com/ethyca/fides/pull/4407)
diff --git a/clients/admin-ui/src/types/api/models/ConsentMethod.ts b/clients/admin-ui/src/types/api/models/ConsentMethod.ts
index 8d0b4dd52c..359761155a 100644
--- a/clients/admin-ui/src/types/api/models/ConsentMethod.ts
+++ b/clients/admin-ui/src/types/api/models/ConsentMethod.ts
@@ -6,7 +6,11 @@
  * An enumeration.
  */
 export enum ConsentMethod {
-  BUTTON = "button",
+  BUTTON = "button", // deprecated- keeping for backwards-compatibility
+  REJECT = "reject",
+  ACCEPT = "accept",
+  SAVE = "save",
+  DISMISS = "dismiss",
   GPC = "gpc",
   INDIVIDUAL_NOTICE = "individual_notice",
 }
diff --git a/clients/fides-js/src/components/ConsentButtons.tsx b/clients/fides-js/src/components/ConsentButtons.tsx
index 06e40d9b95..7e9621ccba 100644
--- a/clients/fides-js/src/components/ConsentButtons.tsx
+++ b/clients/fides-js/src/components/ConsentButtons.tsx
@@ -1,11 +1,12 @@
-import { h, VNode } from "preact";
+import { VNode, h } from "preact";
 import Button from "./Button";
 import {
   ButtonType,
-  PrivacyExperience,
   ConsentMechanism,
-  PrivacyNotice,
+  ConsentMethod,
   ExperienceConfig,
+  PrivacyExperience,
+  PrivacyNotice,
 } from "../lib/consent-types";
 import PrivacyPolicyLink from "./PrivacyPolicyLink";
 
@@ -66,7 +67,7 @@ type NoticeKeys = Array<PrivacyNotice["notice_key"]>;
 
 interface NoticeConsentButtonProps {
   experience: PrivacyExperience;
-  onSave: (noticeKeys: NoticeKeys) => void;
+  onSave: (consentMethod: ConsentMethod, noticeKeys: NoticeKeys) => void;
   onManagePreferencesClick?: () => void;
   enabledKeys: NoticeKeys;
   isAcknowledge: boolean;
@@ -89,11 +90,15 @@ export const NoticeConsentButtons = ({
   const { experience_config: config, privacy_notices: notices } = experience;
 
   const handleAcceptAll = () => {
-    onSave(notices.map((n) => n.notice_key));
+    onSave(
+      ConsentMethod.accept,
+      notices.map((n) => n.notice_key)
+    );
   };
 
   const handleRejectAll = () => {
     onSave(
+      ConsentMethod.reject,
       notices
         .filter((n) => n.consent_mechanism === ConsentMechanism.NOTICE_ONLY)
         .map((n) => n.notice_key)
@@ -101,7 +106,7 @@ export const NoticeConsentButtons = ({
   };
 
   const handleSave = () => {
-    onSave(enabledKeys);
+    onSave(ConsentMethod.save, enabledKeys);
   };
 
   if (isAcknowledge) {
diff --git a/clients/fides-js/src/components/notices/NoticeOverlay.tsx b/clients/fides-js/src/components/notices/NoticeOverlay.tsx
index 6cfb6a8faa..2980814e64 100644
--- a/clients/fides-js/src/components/notices/NoticeOverlay.tsx
+++ b/clients/fides-js/src/components/notices/NoticeOverlay.tsx
@@ -86,7 +86,10 @@ const NoticeOverlay: FunctionComponent<OverlayProps> = ({
   };
 
   const handleUpdatePreferences = useCallback(
-    (enabledPrivacyNoticeKeys: Array<PrivacyNotice["notice_key"]>) => {
+    (
+      consentMethod: ConsentMethod,
+      enabledPrivacyNoticeKeys: Array<PrivacyNotice["notice_key"]>
+    ) => {
       const consentPreferencesToSave = createConsentPreferencesToSave(
         privacyNotices,
         enabledPrivacyNoticeKeys,
@@ -96,7 +99,7 @@ const NoticeOverlay: FunctionComponent<OverlayProps> = ({
       updateConsentPreferences({
         consentPreferencesToSave,
         experience,
-        consentMethod: ConsentMethod.button,
+        consentMethod,
         options,
         userLocationString: fidesRegionString,
         cookie,
@@ -154,8 +157,11 @@ const NoticeOverlay: FunctionComponent<OverlayProps> = ({
               experience={experience}
               onManagePreferencesClick={onManagePreferencesClick}
               enabledKeys={draftEnabledNoticeKeys}
-              onSave={(keys) => {
-                handleUpdatePreferences(keys);
+              onSave={(
+                consentMethod: ConsentMethod,
+                keys: Array<PrivacyNotice["notice_key"]>
+              ) => {
+                handleUpdatePreferences(consentMethod, keys);
                 onSave();
               }}
               isAcknowledge={isAllNoticeOnly}
@@ -183,8 +189,11 @@ const NoticeOverlay: FunctionComponent<OverlayProps> = ({
           <NoticeConsentButtons
             experience={experience}
             enabledKeys={draftEnabledNoticeKeys}
-            onSave={(keys) => {
-              handleUpdatePreferences(keys);
+            onSave={(
+              consentMethod: ConsentMethod,
+              keys: Array<PrivacyNotice["notice_key"]>
+            ) => {
+              handleUpdatePreferences(consentMethod, keys);
               onClose();
             }}
             isInModal
diff --git a/clients/fides-js/src/components/tcf/TcfConsentButtons.tsx b/clients/fides-js/src/components/tcf/TcfConsentButtons.tsx
index 562a661019..d77eb95d1d 100644
--- a/clients/fides-js/src/components/tcf/TcfConsentButtons.tsx
+++ b/clients/fides-js/src/components/tcf/TcfConsentButtons.tsx
@@ -1,13 +1,13 @@
 import { VNode, h } from "preact";
 
-import { PrivacyExperience } from "../../lib/consent-types";
+import { ConsentMethod, PrivacyExperience } from "../../lib/consent-types";
 import { ConsentButtons } from "../ConsentButtons";
 import type { EnabledIds, TcfModels } from "../../lib/tcf/types";
 
 interface TcfConsentButtonProps {
   experience: PrivacyExperience;
   onManagePreferencesClick?: () => void;
-  onSave: (keys: EnabledIds) => void;
+  onSave: (consentMethod: ConsentMethod, keys: EnabledIds) => void;
   firstButton?: VNode;
   isMobile: boolean;
   includePrivacyPolicy?: boolean;
@@ -48,7 +48,7 @@ export const TcfConsentButtons = ({
         ...(experience.tcf_system_legitimate_interests || []),
       ]),
     };
-    onSave(allIds);
+    onSave(ConsentMethod.accept, allIds);
   };
   const handleRejectAll = () => {
     const emptyIds: EnabledIds = {
@@ -60,7 +60,7 @@ export const TcfConsentButtons = ({
       vendorsConsent: [],
       vendorsLegint: [],
     };
-    onSave(emptyIds);
+    onSave(ConsentMethod.reject, emptyIds);
   };
 
   return (
diff --git a/clients/fides-js/src/components/tcf/TcfOverlay.tsx b/clients/fides-js/src/components/tcf/TcfOverlay.tsx
index 5d71d7e354..351c0a527c 100644
--- a/clients/fides-js/src/components/tcf/TcfOverlay.tsx
+++ b/clients/fides-js/src/components/tcf/TcfOverlay.tsx
@@ -1,5 +1,5 @@
-import { h, FunctionComponent, Fragment } from "preact";
-import { useState, useCallback, useMemo } from "preact/hooks";
+import { Fragment, FunctionComponent, h } from "preact";
+import { useCallback, useMemo, useState } from "preact/hooks";
 import ConsentBanner from "../ConsentBanner";
 import PrivacyPolicyLink from "../PrivacyPolicyLink";
 
@@ -18,16 +18,16 @@ import type {
   EnabledIds,
   TCFFeatureRecord,
   TCFFeatureSave,
+  TcfModels,
   TCFPurposeConsentRecord,
   TCFPurposeLegitimateInterestsRecord,
   TCFPurposeSave,
+  TcfSavePreferences,
   TCFSpecialFeatureSave,
   TCFSpecialPurposeSave,
-  TCFVendorSave,
-  TcfSavePreferences,
   TCFVendorConsentRecord,
   TCFVendorLegitimateInterestsRecord,
-  TcfModels,
+  TCFVendorSave,
 } from "../../lib/tcf/types";
 
 import { updateConsentPreferences } from "../../lib/preferences";
@@ -293,7 +293,7 @@ const TcfOverlay: FunctionComponent<OverlayProps> = ({
   });
 
   const handleUpdateAllPreferences = useCallback(
-    (enabledIds: EnabledIds) => {
+    (consentMethod: ConsentMethod, enabledIds: EnabledIds) => {
       const tcf = createTcfSavePayload({
         experience,
         enabledIds,
@@ -302,7 +302,7 @@ const TcfOverlay: FunctionComponent<OverlayProps> = ({
       updateConsentPreferences({
         consentPreferencesToSave: [],
         experience,
-        consentMethod: ConsentMethod.button,
+        consentMethod,
         options,
         userLocationString: fidesRegionString,
         cookie,
@@ -361,8 +361,8 @@ const TcfOverlay: FunctionComponent<OverlayProps> = ({
               <TcfConsentButtons
                 experience={experience}
                 onManagePreferencesClick={onManagePreferencesClick}
-                onSave={(keys) => {
-                  handleUpdateAllPreferences(keys);
+                onSave={(consentMethod: ConsentMethod, keys: EnabledIds) => {
+                  handleUpdateAllPreferences(consentMethod, keys);
                   onSave();
                 }}
                 isMobile={isMobile}
@@ -394,8 +394,8 @@ const TcfOverlay: FunctionComponent<OverlayProps> = ({
         />
       )}
       renderModalFooter={({ onClose, isMobile }) => {
-        const onSave = (keys: EnabledIds) => {
-          handleUpdateAllPreferences(keys);
+        const onSave = (consentMethod: ConsentMethod, keys: EnabledIds) => {
+          handleUpdateAllPreferences(consentMethod, keys);
           onClose();
         };
         return (
@@ -407,7 +407,7 @@ const TcfOverlay: FunctionComponent<OverlayProps> = ({
                 <Button
                   buttonType={ButtonType.SECONDARY}
                   label={experience.experience_config?.save_button_label}
-                  onClick={() => onSave(draftIds)}
+                  onClick={() => onSave(ConsentMethod.save, draftIds)}
                   className="fides-save-button"
                 />
               }
diff --git a/clients/fides-js/src/lib/consent-types.ts b/clients/fides-js/src/lib/consent-types.ts
index e2d0a723bf..ed6586c6a4 100644
--- a/clients/fides-js/src/lib/consent-types.ts
+++ b/clients/fides-js/src/lib/consent-types.ts
@@ -96,11 +96,13 @@ export type FidesApiOptions = {
   /**
    * Intake a custom function that is called instead of the internal Fides API to save user preferences.
    *
+   * @param {object} consentMethod - method that was used to obtain consent
    * @param {object} consent - updated version of Fides.consent with the user's saved preferences for Fides notices
    * @param {string} fides_string - updated version of Fides.fides_string with the user's saved preferences for TC/AC/etc notices
    * @param {object} experience - current version of the privacy experience that was shown to the user
    */
   savePreferencesFn?: (
+    consentMethod: ConsentMethod,
     consent: CookieKeyConsent,
     fides_string: string | undefined,
     experience: PrivacyExperience
@@ -398,7 +400,11 @@ export enum ButtonType {
 }
 
 export enum ConsentMethod {
-  button = "button",
+  button = "button", // deprecated- keeping for backwards-compatibility
+  reject = "reject",
+  accept = "accept",
+  save = "save",
+  dismiss = "dismiss",
   gpc = "gpc",
   individual_notice = "api",
 }
diff --git a/clients/fides-js/src/lib/events.ts b/clients/fides-js/src/lib/events.ts
index 4c0dd295a8..d0ff6b4512 100644
--- a/clients/fides-js/src/lib/events.ts
+++ b/clients/fides-js/src/lib/events.ts
@@ -71,7 +71,11 @@ export const dispatchFidesEvent = (
         extraDetails?.servingComponent
           ? `from ${extraDetails.servingComponent} `
           : ""
-      }with cookie ${JSON.stringify(cookie)}`
+      }with cookie ${JSON.stringify(cookie)} ${
+        extraDetails?.consentMethod
+          ? `using consent method ${extraDetails.consentMethod} `
+          : ""
+      }`
     );
     window.dispatchEvent(event);
   }
diff --git a/clients/fides-js/src/lib/preferences.ts b/clients/fides-js/src/lib/preferences.ts
index b54a58354e..9c2353eb67 100644
--- a/clients/fides-js/src/lib/preferences.ts
+++ b/clients/fides-js/src/lib/preferences.ts
@@ -45,6 +45,7 @@ async function savePreferencesApi(
     ...(tcf ?? []),
   };
   await patchUserPreference(
+    consentMethod,
     privacyPreferenceCreate,
     options,
     cookie,
@@ -130,5 +131,5 @@ export const updateConsentPreferences = async ({
   }
 
   // 6. Dispatch a "FidesUpdated" event
-  dispatchFidesEvent("FidesUpdated", cookie, options.debug);
+  dispatchFidesEvent("FidesUpdated", cookie, options.debug, { consentMethod });
 };
diff --git a/clients/fides-js/src/services/api.ts b/clients/fides-js/src/services/api.ts
index 51f98e8d3f..01bb204031 100644
--- a/clients/fides-js/src/services/api.ts
+++ b/clients/fides-js/src/services/api.ts
@@ -1,5 +1,6 @@
 import {
   ComponentType,
+  ConsentMethod,
   EmptyExperience,
   FidesApiOptions,
   FidesOptions,
@@ -114,6 +115,7 @@ const PATCH_FETCH_OPTIONS: RequestInit = {
  * Sends user consent preference downstream to Fides or custom API
  */
 export const patchUserPreference = async (
+  consentMethod: ConsentMethod,
   preferences: PrivacyPreferencesRequest,
   options: FidesOptions,
   cookie: FidesCookie,
@@ -124,6 +126,7 @@ export const patchUserPreference = async (
     debugLog(options.debug, "Calling custom save preferences fn");
     try {
       await options.apiOptions.savePreferencesFn(
+        consentMethod,
         cookie.consent,
         cookie.fides_string,
         experience
diff --git a/clients/privacy-center/cypress/e2e/consent-banner-tcf.cy.ts b/clients/privacy-center/cypress/e2e/consent-banner-tcf.cy.ts
index 37d8b5baca..d8da84ef1e 100644
--- a/clients/privacy-center/cypress/e2e/consent-banner-tcf.cy.ts
+++ b/clients/privacy-center/cypress/e2e/consent-banner-tcf.cy.ts
@@ -1,6 +1,7 @@
 /* eslint-disable no-underscore-dangle */
 import {
   CONSENT_COOKIE_NAME,
+  ConsentMethod,
   FidesCookie,
   FidesEndpointPaths,
   PrivacyExperience,
@@ -322,7 +323,12 @@ describe("Fides-js TCF", () => {
         cy.getByTestId("consent-modal").within(() => {
           cy.get("button").contains("Opt in to all").click();
         });
-        cy.get("@FidesUpdated").should("have.been.calledOnce");
+        cy.get("@FidesUpdated")
+          .should("have.been.calledOnce")
+          .its("lastCall.args.0.detail.extraDetails.consentMethod")
+          .then((consentMethod) => {
+            expect(consentMethod).to.eql(ConsentMethod.accept);
+          });
         cy.window().then((win) => {
           win.__tcfapi("getTCData", 2, cy.stub().as("getTCData"));
           cy.get("@getTCData")
@@ -514,6 +520,7 @@ describe("Fides-js TCF", () => {
           cy.wait("@patchPrivacyPreference").then((interception) => {
             cy.get("@FidesUIChanged").should("not.have.been.called");
             const { body } = interception.request;
+            expect(body.method).to.eql(ConsentMethod.accept);
             expect(body.purpose_consent_preferences).to.eql([
               {
                 id: PURPOSE_4.id,
@@ -620,6 +627,9 @@ describe("Fides-js TCF", () => {
           cy.wait("@patchPrivacyPreference").then((interception) => {
             cy.get("@FidesUIChanged").should("not.have.been.called");
             const { body } = interception.request;
+            expect(interception.request.body.method).to.eql(
+              ConsentMethod.reject
+            );
             expect(body.purpose_consent_preferences).to.eql([
               {
                 id: PURPOSE_4.id,
@@ -738,6 +748,7 @@ describe("Fides-js TCF", () => {
           cy.get("@FidesUIChanged").its("callCount").should("equal", 3);
           cy.wait("@patchPrivacyPreference").then((interception) => {
             const { body } = interception.request;
+            expect(interception.request.body.method).to.eql(ConsentMethod.save);
             expect(body.purpose_consent_preferences).to.eql([
               {
                 id: PURPOSE_4.id,
@@ -841,11 +852,12 @@ describe("Fides-js TCF", () => {
       it("calls custom save preferences API fn instead of internal Fides API when it is provided in Fides.init", () => {
         const apiOptions = {
           /* eslint-disable @typescript-eslint/no-unused-vars */
-          savePreferencesFn: (
+          savePreferencesFn: async (
+            consentMethod: ConsentMethod,
             consent: CookieKeyConsent,
             fides_string: string | undefined,
             experience: PrivacyExperience
-          ): Promise<void> => new Promise(() => {}),
+          ): Promise<void> => {},
           /* eslint-enable @typescript-eslint/no-unused-vars */
         };
         const spyObject = cy
@@ -864,31 +876,36 @@ describe("Fides-js TCF", () => {
             cy.get("#fides-modal-link").click();
             cy.getByTestId("consent-modal").within(() => {
               cy.get("button").contains("Opt out of all").click();
-              cy.get("@FidesUpdated").then(() => {
-                // eslint-disable-next-line @typescript-eslint/no-unused-expressions
-                expect(spyObject).to.be.called;
-                const spy = spyObject.getCalls();
-                const { args } = spy[0];
-                expect(args[0]).to.deep.equal({
-                  data_sales: true,
-                  tracking: false,
+              cy.get("@FidesUpdated")
+                .should("have.been.calledOnce")
+                .its("lastCall.args.0.detail.extraDetails.consentMethod")
+                .then((consentMethod) => {
+                  expect(consentMethod).to.eql(ConsentMethod.reject);
+                  // eslint-disable-next-line @typescript-eslint/no-unused-expressions
+                  expect(spyObject).to.be.called;
+                  const spy = spyObject.getCalls();
+                  const { args } = spy[0];
+                  expect(args[0]).to.equal(ConsentMethod.reject);
+                  expect(args[1]).to.deep.equal({
+                    data_sales: true,
+                    tracking: false,
+                  });
+                  // the TC str is dynamically updated upon save preferences with diff timestamp, so we do a fuzzy match
+                  expect(args[2]).to.contain(".IABE,1~");
+                  expect(args[3]).to.deep.equal(privacyExperience.items[0]);
+                  // timeout means API call not made, which is expected
+                  cy.on("fail", (error) => {
+                    if (error.message.indexOf("Timed out retrying") !== 0) {
+                      throw error;
+                    }
+                  });
+                  // check that preferences aren't sent to Fides API
+                  cy.wait("@patchPrivacyPreference", {
+                    requestTimeout: 100,
+                  }).then((xhr) => {
+                    assert.isNull(xhr?.response?.body);
+                  });
                 });
-                // the TC str is dynamically updated upon save preferences with diff timestamp, so we do a fuzzy match
-                expect(args[1]).to.contain(".IABE,1~");
-                expect(args[2]).to.deep.equal(privacyExperience.items[0]);
-              });
-              // timeout means API call not made, which is expected
-              cy.on("fail", (error) => {
-                if (error.message.indexOf("Timed out retrying") !== 0) {
-                  throw error;
-                }
-              });
-              // check that preferences aren't sent to Fides API
-              cy.wait("@patchPrivacyPreference", {
-                requestTimeout: 100,
-              }).then((xhr) => {
-                assert.isNull(xhr?.response?.body);
-              });
             });
           });
         });
@@ -1133,6 +1150,7 @@ describe("Fides-js TCF", () => {
       cy.get("button").contains("Save").click();
       cy.wait("@patchPrivacyPreference").then((interception) => {
         const { body } = interception.request;
+        expect(interception.request.body.method).to.eql(ConsentMethod.save);
         expect(body.purpose_consent_preferences).to.eql([
           { id: PURPOSE_4.id, preference: "opt_out" },
           { id: PURPOSE_6.id, preference: "opt_in" },
@@ -1284,7 +1302,12 @@ describe("Fides-js TCF", () => {
           cy.get("button").contains("Opt in to all").click();
         });
         // On slow connections, we should explicitly wait for FidesUpdated
-        cy.get("@FidesUpdated").should("have.been.calledOnce");
+        cy.get("@FidesUpdated")
+          .should("have.been.calledOnce")
+          .its("lastCall.args.0.detail.extraDetails.consentMethod")
+          .then((consentMethod) => {
+            expect(consentMethod).to.eql(ConsentMethod.accept);
+          });
         cy.get("@TCFEvent")
           .its("lastCall.args")
           .then(([tcData, success]) => {
@@ -1348,7 +1371,12 @@ describe("Fides-js TCF", () => {
         cy.getByTestId("consent-modal").within(() => {
           cy.get("button").contains("Opt in to all").click();
         });
-        cy.get("@FidesUpdated").should("have.been.calledOnce");
+        cy.get("@FidesUpdated")
+          .should("have.been.calledOnce")
+          .its("lastCall.args.0.detail.extraDetails.consentMethod")
+          .then((consentMethod) => {
+            expect(consentMethod).to.eql(ConsentMethod.accept);
+          });
         cy.get("@TCFEvent2")
           .its("lastCall.args")
           .then(([tcData, success]) => {
@@ -2526,6 +2554,7 @@ describe("Fides-js TCF", () => {
           ...AC_IDS.map((id) => ({ id: `gacp.${id}`, preference: "opt_in" })),
         ];
         expect(body.vendor_consent_preferences).to.eql(expected);
+        expect(body.method).to.eql(ConsentMethod.save);
 
         // Check the cookie
         cy.waitUntilCookieExists(CONSENT_COOKIE_NAME).then(() => {
@@ -2553,6 +2582,7 @@ describe("Fides-js TCF", () => {
           ...AC_IDS.map((id) => ({ id: `gacp.${id}`, preference: "opt_out" })),
         ];
         expect(body.vendor_consent_preferences).to.eql(expected);
+        expect(body.method).to.eql(ConsentMethod.reject);
 
         // Check the cookie
         cy.waitUntilCookieExists(CONSENT_COOKIE_NAME).then(() => {
@@ -2576,7 +2606,9 @@ describe("Fides-js TCF", () => {
       cy.getByTestId("consent-modal").within(() => {
         cy.get("button").contains("Opt in to all").click();
       });
-      cy.wait("@patchPrivacyPreference");
+      cy.wait("@patchPrivacyPreference").then((interception) => {
+        expect(interception.request.body.method).to.eql(ConsentMethod.accept);
+      });
       cy.get("@FidesUpdated")
         .should("have.been.calledOnce")
         .its("lastCall.args.0.detail.fides_string")
@@ -2606,8 +2638,15 @@ describe("Fides-js TCF", () => {
       cy.getByTestId("consent-modal").within(() => {
         cy.get("button").contains("Opt in to all").click();
       });
-      cy.wait("@patchPrivacyPreference");
-      cy.get("@FidesUpdated").should("have.been.calledOnce");
+      cy.wait("@patchPrivacyPreference").then((interception) => {
+        expect(interception.request.body.method).to.eql(ConsentMethod.accept);
+      });
+      cy.get("@FidesUpdated")
+        .should("have.been.calledOnce")
+        .its("lastCall.args.0.detail.extraDetails.consentMethod")
+        .then((consentMethod) => {
+          expect(consentMethod).to.eql(ConsentMethod.accept);
+        });
       // Call getTCData
       cy.window().then((win) => {
         win.__tcfapi("getTCData", 2, cy.stub().as("getTCData"));
diff --git a/clients/privacy-center/cypress/e2e/consent-banner.cy.ts b/clients/privacy-center/cypress/e2e/consent-banner.cy.ts
index 36a0c54a07..c0fefeccdb 100644
--- a/clients/privacy-center/cypress/e2e/consent-banner.cy.ts
+++ b/clients/privacy-center/cypress/e2e/consent-banner.cy.ts
@@ -204,7 +204,7 @@ describe("Consent banner", () => {
               ],
               privacy_experience_id: "132345243",
               user_geography: "us_ca",
-              method: ConsentMethod.button,
+              method: ConsentMethod.save,
             };
             // uuid is generated automatically if the user has no saved consent cookie
             generatedUserDeviceId = body.browser_identity.fides_user_device_id;
@@ -336,7 +336,7 @@ describe("Consent banner", () => {
             ],
             privacy_experience_id: "132345243",
             user_geography: "us_ca",
-            method: ConsentMethod.button,
+            method: ConsentMethod.save,
           };
           expect(body).to.eql(expected);
         });
@@ -409,7 +409,12 @@ describe("Consent banner", () => {
 
         it("can remove all cookies when rejecting all", () => {
           cy.contains("button", "Reject Test").click();
-          cy.get("@FidesUpdated").should("have.been.calledOnce");
+          cy.get("@FidesUpdated")
+            .should("have.been.calledOnce")
+            .its("lastCall.args.0.detail.extraDetails.consentMethod")
+            .then((consentMethod) => {
+              expect(consentMethod).to.eql(ConsentMethod.reject);
+            });
           cy.getAllCookies().then((allCookies) => {
             expect(allCookies.map((c) => c.name)).to.eql([CONSENT_COOKIE_NAME]);
           });
@@ -420,7 +425,12 @@ describe("Consent banner", () => {
           // opt out of the first notice
           cy.getByTestId("toggle-one").click();
           cy.getByTestId("Save test-btn").click();
-          cy.get("@FidesUpdated").should("have.been.calledOnce");
+          cy.get("@FidesUpdated")
+            .should("have.been.calledOnce")
+            .its("lastCall.args.0.detail.extraDetails.consentMethod")
+            .then((consentMethod) => {
+              expect(consentMethod).to.eql(ConsentMethod.save);
+            });
           cy.getAllCookies().then((allCookies) => {
             expect(allCookies.map((c) => c.name)).to.eql([
               CONSENT_COOKIE_NAME,
@@ -1172,6 +1182,11 @@ describe("Consent banner", () => {
             [PRIVACY_NOTICE_KEY_1]: false,
             [PRIVACY_NOTICE_KEY_2]: true,
           });
+        cy.get("@FidesUpdated")
+          .its("lastCall.args.0.detail.extraDetails.consentMethod")
+          .then((consentMethod) => {
+            expect(consentMethod).to.eql(ConsentMethod.reject);
+          });
       });
 
       it("emits a FidesUpdated event when accept all is clicked", () => {
@@ -1193,6 +1208,11 @@ describe("Consent banner", () => {
             [PRIVACY_NOTICE_KEY_1]: true,
             [PRIVACY_NOTICE_KEY_2]: true,
           });
+        cy.get("@FidesUpdated")
+          .its("lastCall.args.0.detail.extraDetails.consentMethod")
+          .then((consentMethod) => {
+            expect(consentMethod).to.eql(ConsentMethod.accept);
+          });
       });
 
       it("emits a FidesUIChanged event when preferences are changed and a FidesUpdated event when preferences are saved", () => {
@@ -1210,6 +1230,7 @@ describe("Consent banner", () => {
             [PRIVACY_NOTICE_KEY_1]: false,
             [PRIVACY_NOTICE_KEY_2]: true,
           });
+
         cy.get("@FidesUpdated")
           // Update event, when the user saved preferences and opted-in to the first notice
           .should("have.been.calledOnce")
@@ -1218,6 +1239,11 @@ describe("Consent banner", () => {
             [PRIVACY_NOTICE_KEY_1]: true,
             [PRIVACY_NOTICE_KEY_2]: true,
           });
+        cy.get("@FidesUpdated")
+          .its("lastCall.args.0.detail.extraDetails.consentMethod")
+          .then((consentMethod) => {
+            expect(consentMethod).to.eql(ConsentMethod.save);
+          });
       });
     });
 
@@ -1586,6 +1612,9 @@ describe("Consent banner", () => {
               (p: ConsentOptionCreate) => p.served_notice_history_id
             )
           ).to.eql(expected);
+          expect(preferenceInterception.request.body.method).to.eql(
+            ConsentMethod.reject
+          );
         });
       });
     });
diff --git a/src/fides/api/models/privacy_preference.py b/src/fides/api/models/privacy_preference.py
index 130bbcc3d6..a53f344bdf 100644
--- a/src/fides/api/models/privacy_preference.py
+++ b/src/fides/api/models/privacy_preference.py
@@ -52,7 +52,11 @@ class RequestOrigin(Enum):
 
 
 class ConsentMethod(Enum):
-    button = "button"
+    button = "button"  # deprecated- keeping for backwards-compatibility
+    reject = "reject"
+    accept = "accept"
+    save = "save"
+    dismiss = "dismiss"
     gpc = "gpc"
     individual_notice = "individual_notice"