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.

Sign up for GitHub

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

Jump to bottom

"max event-level reports per attribution destination" check may be wrong with respect to report replacement #1267

Closed
apasel422 opened this issue May 2, 2024 · 1 comment · Fixed by #1268
Closed

"max event-level reports per attribution destination" check may be wrong with respect to report replacement #1267

apasel422 opened this issue May 2, 2024 · 1 comment · Fixed by #1268
Assignees
@johnivdel
Labels
bug Something isn't working documentation

Comments

@apasel422
Copy link
Collaborator

This check happens before we determine whether the new trigger will cause an existing report to be replaced, in which case the number of reports per attribution destination may remain the same but the trigger to be rejected.
@apasel422 apasel422 added bug Something isn't working documentation labels May 2, 2024
@apasel422
Copy link
Collaborator Author

CC @linnan-github

@apasel422 apasel422 mentioned this issue May 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnivdel johnivdel

Labels
bug Something isn't working documentation
Projects
None yet
Milestone
No milestone
2 participants
@apasel422 @johnivdel