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

feat(bright-cli): track command usage #564

Merged
merged 11 commits into from
Sep 2, 2024

Conversation

bramkor
Copy link
Contributor

@bramkor bramkor commented Jul 12, 2024

Track command usage by intercepting every event and send it to analytics-service endpoint.

@bramkor bramkor self-assigned this Jul 12, 2024
@bramkor bramkor added the Type: enhancement New feature or request. label Jul 12, 2024
@bramkor bramkor marked this pull request as ready for review August 15, 2024 12:59
@bramkor bramkor requested a review from maksadbek August 15, 2024 12:59
src/Utils/Tracker.ts Outdated Show resolved Hide resolved
bram kortooms added 2 commits August 16, 2024 12:43
Add middleware to track command usage, and send events to analytics api.

PAP-1083 #code-review
…Legion/bright-cli into feat-#PAP-1083/track-command-usage
@bramkor bramkor requested a review from maksadbek August 16, 2024 10:44
maksadbek
maksadbek previously approved these changes Aug 16, 2024
Copy link
Collaborator

@maksadbek maksadbek left a comment

Choose a reason for hiding this comment

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

lgtm

@bramkor bramkor changed the base branch from master to next August 16, 2024 11:29
@bramkor bramkor dismissed maksadbek’s stale review August 16, 2024 11:29

The base branch was changed.

@bramkor bramkor requested a review from maksadbek August 19, 2024 07:58
@maksadbek maksadbek merged commit 320cce8 into next Sep 2, 2024
6 checks passed
@maksadbek maksadbek deleted the feat-#PAP-1083/track-command-usage branch September 2, 2024 11:02
bramkor added a commit that referenced this pull request Sep 5, 2024
orubin pushed a commit that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: enhancement New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants