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

[Stratconn-2879] | Set up DataDog dashboard to track requests to Oauth Supported Destinations. #1585

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

Innovative-GauravKochar
Copy link
Contributor

@Innovative-GauravKochar Innovative-GauravKochar commented Sep 18, 2023

This Pull request is to set up DataDog dashboard to track requests to Oauth Supported Destinations.
Jira Ticket:- https://segment.atlassian.net/jira/software/c/projects/STRATCONN/boards/310?modal=detail&selectedIssue=STRATCONN-2879&assignee=63617339fc0cc7a600b03c6b

Testing

Include any additional information about the testing you have completed to
ensure your changes behave as expected. For a speedy review, please check
any of the tasks you completed below during your testing.

  • Added unit tests for new functionality - Not Required
  • Tested end-to-end using the local server - Not Required
  • [Segmenters] Tested in the staging environment -

@Innovative-GauravKochar Innovative-GauravKochar marked this pull request as ready for review September 25, 2023 12:46
@brennan
Copy link
Contributor

brennan commented Sep 25, 2023

@Innovative-GauravKochar Thank you for putting all this together! After seeing how many changes oauth metrics tracking entails, I think I'd prefer to extract LinkedIn into its own PR and merge/deploy that first so we limit the blast radius of the change. Once we get signal on how useful metrics are for LinkedIn, then we can move forward with adding metrics to the other destinations.

@Innovative-GauravKochar is it possible to set up a PR with only the changes to LinkedIn sometime this week? Thank you again!

@Innovative-GauravKochar
Copy link
Contributor Author

@brennan Created a Seperate PR for linkedin-audiences #1619

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants