Skip to content

Commit

Permalink
fix: confirming cookie preferences removes punchout related cookies (#…
Browse files Browse the repository at this point in the history
…1619)

* punchout cookies have to be added to the list of required cookies
* if punchout cookies are removed with the cookies consent the punchout functionality will no longer work as expected
* added to documentation as well
  • Loading branch information
shauke authored Mar 28, 2024
1 parent 2e84cb7 commit d929606
Show file tree
Hide file tree
Showing 4 changed files with 27 additions and 7 deletions.
Binary file modified docs/guides/cookie-banner.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
24 changes: 18 additions & 6 deletions docs/guides/cookie-consent.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,15 @@ cookieConsentOptions: {
description: 'cookie.consent.option.tracking.description',
},
},
allowedCookies: ['apiToken', 'cookieConsent', 'preferredLocale'],
allowedCookies: [
'apiToken',
'cookieConsent',
'preferredLocale',
'punchout_SID',
'punchout_BasketID',
'punchout_ReturnURL',
'punchout_HookURL',
],
},
```

Expand Down Expand Up @@ -100,11 +108,15 @@ This route can be linked to from anywhere within the application.

## PWA Required Cookies

| Name | Expiration | Provider | Description | Category |
| --------------- | ---------- | ------------- | ----------------------------------------------------------------- | ----------- |
| apiToken | 1 year | Intershop PWA | The API token used by the Intershop Commerce Management REST API. | First Party |
| cookieConsent | 1 year | Intershop PWA | Saves the user's cookie consent settings. | First Party |
| preferredLocale | 1 year | Intershop PWA | Saves the user's language selection. | First Party |
| Name | Expiration | Provider | Description | Category |
| ------------------ | ---------- | ------------- | ----------------------------------------------------------------- | ----------- |
| apiToken | 1 hour | Intershop PWA | The API token used by the Intershop Commerce Management REST API. | First Party |
| cookieConsent | 1 year | Intershop PWA | Saves the user's cookie consent settings. | First Party |
| preferredLocale | 1 year | Intershop PWA | Saves the user's language selection. | First Party |
| punchout_SID | Session | Intershop PWA | Saves punchout session related data - Session ID. | First Party |
| punchout_BasketID | Session | Intershop PWA | Saves punchout session related data - Basket ID. | First Party |
| punchout_ReturnURL | Session | Intershop PWA | Saves punchout session related data - Return URL. | First Party |
| punchout_HookURL | Session | Intershop PWA | Saves punchout session related data - Hook URL. | First Party |

## Disabling the Integrated Cookie Consent Handling

Expand Down
Binary file modified docs/guides/cookie-preferences.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
10 changes: 9 additions & 1 deletion src/environments/environment.model.ts
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,15 @@ export const ENVIRONMENT_DEFAULTS: Omit<Environment, 'icmChannel'> = {
description: 'cookie.consent.option.tracking.description',
},
},
allowedCookies: ['apiToken', 'cookieConsent', 'preferredLocale'],
allowedCookies: [
'apiToken',
'cookieConsent',
'preferredLocale',
'punchout_SID',
'punchout_BasketID',
'punchout_ReturnURL',
'punchout_HookURL',
],
},
cookieConsentVersion: 1,

Expand Down

0 comments on commit d929606

Please sign in to comment.