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

Modified the signature of ITelemetryLogger #813

Merged
merged 42 commits into from
Jan 19, 2024
Merged

Modified the signature of ITelemetryLogger #813

merged 42 commits into from
Jan 19, 2024

Conversation

BidishaMS
Copy link
Contributor

@BidishaMS BidishaMS commented Jan 17, 2024 •
edited
Loading

Modified the signature of ITelemetryLogger

  1. Renamed from customDimension to eventInfo for all the APIs
  2. modified signature of traceError
  3. Added featureUsage API
  4. Added few checks for redacting error traces message

BidishaMS and others added 30 commits January 4, 2024 13:25
@BidishaMS
First cut of telemetry
cb8fee0
@BidishaMS
First cut of telemetry
1d65e5d
@BidishaMS
changes in 1DSCollector
39c424b
@BidishaMS
Merge branch 'main' into users/biddas/clientLoggerWrapper
5cf3f79
@BidishaMS
Merge branch 'main' into users/biddas/clientLoggerWrapper
5c99e6a
@BidishaMS
renamed from collector to logger
faecb48
@BidishaMS
Merge branch 'users/biddas/clientLoggerWrapper' of https://github.com…
7c65e72 
…/microsoft/powerplatform-vscode into users/biddas/clientLoggerWrapper
@BidishaMS
merged to remote
e9058b4
@BidishaMS
Merge branch 'main' into users/biddas/clientLoggerWrapper
cf47caf
@BidishaMS
Added wrapper
10975c5
@BidishaMS
consumed in client
6a3f4dc
@BidishaMS
added types
9392543
@BidishaMS
merged master
fc9f264
@BidishaMS
changes
1717d5d
@BidishaMS
Created common interfaces and wrapper
32ace97
@BidishaMS
reverted back changes
59da8c3
@BidishaMS
removed blank line
4628533
@BidishaMS
removed blank line
3f745bf
@BidishaMS
removed blank line
2fec1e4
@BidishaMS
Merge branch 'main' into users/biddas/clientLoggerWrapper1
61c5afb
@BidishaMS
Merge branch 'main' into users/biddas/clientLoggerWrapper1
ee6954c
@BidishaMS
populate data
6775fd6
@BidishaMS
Removed platformType
438eb9d
@BidishaMS
Merge branch 'main' of https://github.com/microsoft/powerplatform-vscode
bd86c55 
 into users/biddas/clientLoggerWrapper1
@BidishaMS
Merge branch 'users/biddas/clientLoggerWrapper1' of https://github.co…
a22e26e 
…m/microsoft/powerplatform-vscode into users/biddas/clientLoggerWrapper1
@BidishaMS
Merge branch 'users/biddas/clientLoggerWrapper1' into users/biddas/Po…
f7d35f8 
…pulateData
@BidishaMS
Removed platformType
f470dc2
@BidishaMS
changes
fab0f38
@BidishaMS
changes
3e46ef9
@BidishaMS
changes
bfc06ef
@BidishaMS BidishaMS changed the title Users/biddas/modify ap is [Draft] Users/biddas/modify ap is Jan 17, 2024
@BidishaMS BidishaMS changed the title [Draft] Users/biddas/modify ap is Modified the signature of ITelemetryLogger Jan 19, 2024
@BidishaMS BidishaMS marked this pull request as ready for review January 19, 2024 11:55
@BidishaMS BidishaMS requested a review from a team as a code owner January 19, 2024 11:55
@BidishaMS BidishaMS enabled auto-merge (squash) January 19, 2024 11:56
@BidishaMS BidishaMS merged commit dfa8aa4 into main Jan 19, 2024
6 checks passed
@BidishaMS BidishaMS deleted the users/biddas/ModifyAPIs branch January 19, 2024 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tehcrashxor tehcrashxor tehcrashxor approved these changes

@amitjoshi438 amitjoshi438 amitjoshi438 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BidishaMS @tehcrashxor @amitjoshi438