Skip to content

Referrer cookie refactor front#5943

Ben requested to merge feat/referrer-cookie-f5943 into master

Ticket(s) / Related Merge Requests

front#5943 (closed)

Summary of Changes

Refactor to referral cookie logic.

Testing Considerations

Running through testing for the AC, item by item:

Referral cookies will be overwritten by the last referral link visited

Visit any URL with ?referrer=<<any username>> at the end of the URL. If you look in your storage you should now have a cookie for referrer with a matching username.

Visit any URL with the referrer query param with a different user - value in Cookie should be replaced.

Referral cookies will expire after 72 hours

In your dev tools storage panel, the Cookie expiry time should be 72 hours from now.

Referral cookies should work for existing logged in users

Test the cookie is settable when logged in and is sent with requests when logged in.

Integration tests for ensuring the cookie logic is as above

Check the new Referrer tests pass in the CI. There IS a known failure around BlockchainRestrictions - there is a card to fix this sprint.

Deployment Considerations

Standard deployment.

Regression Scope

Referrer cookie.

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

Unit (PHPSpec), Integration (Codeception), Manual

Screenshots / Screen Recording

simplescreenrecorder-2023-03-28_13.19.19

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports