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

[editor] Starting Point for Telemetry #804

Closed
wants to merge 1 commit into from
Closed

[editor] Starting Point for Telemetry #804

wants to merge 1 commit into from

Conversation

rholinshead
Copy link
Contributor

@rholinshead rholinshead commented Jan 6, 2024

[editor] Starting Point for Telemetry

Add the required configuration for datadog browser logging to our local editor. Pass this through as logEvent so that other deployments can handle logging of supported events their own way.

Just added a couple event for now & set logEvent in context for use by any component under the EditorContainer.

Testing:

DEV: aiconfig edit --aiconfig-path=python/src/aiconfig/editor/travel.aiconfig.json --server-mode='debug_servers'

  • Add a prompt and manually save. Make sure the logging shows up with correct env:
Screenshot 2024-01-06 at 5 26 50 PM

PROD: aiconfig edit --aiconfig-path=python/src/aiconfig/editor/travel.aiconfig.json --server-mode='prod'

  • Add a prompt and manually save. Make sure the logging shows up with correct env:
Screenshot 2024-01-06 at 5 30 57 PM

Ankush-lastmile pushed a commit that referenced this pull request Jan 11, 2024
rebased version #804 onto main
Ankush-lastmile pushed a commit that referenced this pull request Jan 11, 2024
rebased version of #804 onto main
Ankush-lastmile pushed a commit that referenced this pull request Jan 13, 2024
rebased version of #804 onto main
@rholinshead
Copy link
Contributor Author

Closing in favour of #864

Ankush-lastmile pushed a commit that referenced this pull request Jan 14, 2024
rebased version of #804 onto main
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