-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
dotnet userfeedback #13794
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
dotnet userfeedback #13794
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
Bundle ReportChanges will decrease total bundle size by 15 bytes (-0.0%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
|
Support for the new Feedback APIs was added here: The original PR has two different overloads: Another overload allowing for a ConfigureScope parameter was added in this PR: |
Co-authored-by: James Crosswell <jamescrosswell@users.noreply.github.com>
|
Thanks James |
We should do a rename on all of the docs for `CaptureUserFeedback` since the API is deprecated. @jamescrosswell does this overload exist? I quickly looked the API and it takes `UserFeedback` as an argument. Calling `CaptureFeedback(new Feedback` is pretty cumbersome --------- Co-authored-by: James Crosswell <jamescrosswell@users.noreply.github.com>
We should do a rename on all of the docs for
CaptureUserFeedbacksince the API is deprecated.@jamescrosswell does this overload exist? I quickly looked the API and it takes
UserFeedbackas an argument.Calling
CaptureFeedback(new Feedbackis pretty cumbersome