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

Allow goals being triggered when a visitor visits a certain number of pages #17466

Open
wants to merge 6 commits into
base: 5.x-dev
Choose a base branch
from

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Apr 16, 2021

Description:

fixes #2029

Review

  • Functional review done
  • Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
  • Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
  • Security review done see checklist
  • Code review done
  • Tests were added if useful/possible
  • Reviewed for breaking changes
  • Developer changelog updated if needed
  • Documentation added if needed
  • Existing documentation updated if needed

core/Tracker/GoalManager.php Outdated Show resolved Hide resolved
@sgiehl sgiehl marked this pull request as ready for review April 16, 2021 13:14
@sgiehl sgiehl added Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. labels Apr 16, 2021
@sgiehl sgiehl added this to the 4.3.0 milestone Apr 16, 2021
@sgiehl sgiehl force-pushed the pageviewgoal branch 2 times, most recently from 2a1533d to ef6cb73 Compare April 22, 2021 14:56
@sgiehl sgiehl modified the milestones: 4.3.0, 5.0.0 May 3, 2021
@sgiehl sgiehl force-pushed the pageviewgoal branch 3 times, most recently from f3c771c to 713135f Compare May 6, 2021 15:23
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Jun 4, 2021
@sgiehl sgiehl added Do not close PRs with this label won't be marked as stale by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Jun 8, 2021
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added Stale The label used by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Jun 16, 2021
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added Stale The label used by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Jun 24, 2021
@sgiehl sgiehl removed the Needs Review PRs that need a code review label Jun 24, 2021
@tsteur tsteur modified the milestones: 5.0.0, 6.0.0 Jul 11, 2022
@tsteur tsteur modified the milestones: 5.0.0, 6.0.0 Jul 18, 2022
@sgiehl sgiehl changed the base branch from 4.x-dev to 5.x-dev November 29, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do not close PRs with this label won't be marked as stale by the Close Stale Issues action not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
4 participants