Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rename cookie and ensure timeout does not fire prematurely #2509

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

brindy
Copy link
Contributor

@brindy brindy commented Feb 28, 2024

Task/Issue URL: https://app.asana.com/0/414235014887631/1206707849446765/f
Tech Design URL:
CC:

Description:

rename cookie and ensure timeout does not fire prematurely

Steps to test this PR:

  1. Use the fire button - the cookie timeout pixel should not fire
  2. Add try? await Task.sleep(interval: 6.0) to the legacyDataClearing() function before the timeout task gets cancelled
  3. Use the fire button - the cookie timeout pixel will fire

@brindy brindy changed the base branch from main to release/7.110.0 February 28, 2024 10:46
@brindy brindy requested a review from quanganhdo February 28, 2024 10:50
Copy link
Member

@quanganhdo quanganhdo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM & tested on my phone 👍

@brindy brindy merged commit 155029e into release/7.110.0 Feb 28, 2024
18 checks passed
@brindy brindy deleted the brindy/fix-pixel-misfire branch February 28, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants