Title: Discussion issue for IFeedbackProvider on Success Noisy Scenarios · Issue #19452 · PowerShell/PowerShell · GitHub
Open Graph Title: Discussion issue for IFeedbackProvider on Success Noisy Scenarios · Issue #19452 · PowerShell/PowerShell
X Title: Discussion issue for IFeedbackProvider on Success Noisy Scenarios · Issue #19452 · PowerShell/PowerShell
Description: Summary of the new feature / enhancement Description When thinking about having the Feedback Provider of PowerShell be able to trigger on success, per #19372, there is a concern around if this will be very noisy and annoying to users. Th...
Open Graph Description: Summary of the new feature / enhancement Description When thinking about having the Feedback Provider of PowerShell be able to trigger on success, per #19372, there is a concern around if this will...
X Description: Summary of the new feature / enhancement Description When thinking about having the Feedback Provider of PowerShell be able to trigger on success, per #19372, there is a concern around if this will...
Opengraph URL: https://github.com/PowerShell/PowerShell/issues/19452
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Discussion issue for IFeedbackProvider on Success Noisy Scenarios","articleBody":"### Summary of the new feature / enhancement\r\n\r\n## Description\r\n\r\nWhen thinking about having the Feedback Provider of PowerShell be able to trigger on success, per #19372, there is a concern around if this will be very noisy and annoying to users. The basic scenario being that a user runs a command successfully but there is potentially a better way or suggested way to enhance their experience, like the JSON adapter proposition in #19372. This could also be a good way to expose capabilities of PowerShell and nuances to help inform the user about potential ramifications.\r\n\r\n## Example\r\nReallocation of array each time for addition:\r\n```powershell\r\n\u003e$a = @()\r\n\u003e$a += 1\r\n\u003e$a += 2\r\n```\r\nThis works and is successful but a more memory conscious approach would be:\r\n\r\n```powershell\r\n\u003e$a = [System.Collections.ArrayList]::new() \r\nor\r\n\u003e$a = [System.Collections.Generic.List[object]]::new()\r\n```\r\nThe feedback provider could suggestion something like \r\n```powershell\r\n\u003e$a = @()\r\n\u003e$a += 1\r\n[suggestion]\r\n $a is being reallocated each time you append to it, consider defining it as a list:\r\n -\u003e $a = [System.Collections.ArrayList]::new() \r\n```\r\n\r\n### Proposed technical implementation details (optional)\r\n\r\n## Questions\r\n\r\nThe scenarios outlined above begs a lot of questions about the experience of Feedback Providers and if its something the PowerShell engine should step in to ensure \"non noisy\" behavior or let the feedback providers themselves be responsible for. For contexts feedback providers are developed as extensible PowerShell modules that can be imported.\r\n\r\n- Should the feedback providers be left responsible for not being \"noisy\" to the users? \r\n \r\nIf left up to the engine:\r\n- How do we classify if something is noisy or not?\r\n- If, after shown at least once after successful, a feedback suggestion is NOT used by the user, then should it be disabled?\r\n- Is there a better way to manage feedback providers besides importing the modules?\r\n- How can we get an indication from the user that feedback is no longer needed on a topic? (i.e dismissal of suggestion)\r\n\r\n\r\nI am sure I am missing some questions or scenarios but want to open an issue to track these and have a discussion around this concept, any and all feedback is appreciated!\r\n\r\nOther related PRs/issues with Feedback Providers\r\n- https://github.com/PowerShell/PowerShell/pull/19133\r\n- #19372 \r\n- #19364\r\n- #18852\r\n- https://github.com/PowerShell/PowerShell/issues/19524\r\n- https://github.com/PowerShell/command-not-found\r\n\r\ncc @daxian-dbw @SteveL-MSFT @JamesWTruher ","author":{"url":"https://github.com/StevenBucher98","@type":"Person","name":"StevenBucher98"},"datePublished":"2023-04-05T00:16:05.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/19452/PowerShell/issues/19452"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:a9cb4878-de68-595b-5cf5-054c80d30cee |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 8978:6E56C:46B079D:626F786:6A54E740 |
| html-safe-nonce | ec2996ac8fa6f3744a85d4dcca962ffcb3e0fe78a9434748893adb3e5b1c5d39 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4OTc4OjZFNTZDOjQ2QjA3OUQ6NjI2Rjc4Njo2QTU0RTc0MCIsInZpc2l0b3JfaWQiOiI5MTEyMjI5NzE4MjIxMTg2ODgwIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | e90605436d14c7a82ab3e63cc29b784f069e95449175eb68fccf16fad404a132 |
| hovercard-subject-tag | issue:1654740207 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/PowerShell/PowerShell/19452/issue_layout |
| twitter:image | https://opengraph.githubassets.com/a9827dc7e01c11c8ab3fd10d78b235150042c61a506f28dec44590aa32b01fab/PowerShell/PowerShell/issues/19452 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/a9827dc7e01c11c8ab3fd10d78b235150042c61a506f28dec44590aa32b01fab/PowerShell/PowerShell/issues/19452 |
| og:image:alt | Summary of the new feature / enhancement Description When thinking about having the Feedback Provider of PowerShell be able to trigger on success, per #19372, there is a concern around if this will... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | StevenBucher98 |
| hostname | github.com |
| expected-hostname | github.com |
| None | eae2aab99907699a22d2c8449aa683ca74454fca87211b98f7c7b722086b2839 |
| turbo-cache-control | no-preview |
| go-import | github.com/PowerShell/PowerShell git https://github.com/PowerShell/PowerShell.git |
| octolytics-dimension-user_id | 11524380 |
| octolytics-dimension-user_login | PowerShell |
| octolytics-dimension-repository_id | 49609581 |
| octolytics-dimension-repository_nwo | PowerShell/PowerShell |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 49609581 |
| octolytics-dimension-repository_network_root_nwo | PowerShell/PowerShell |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | bb1b4ffdc07c66de20ad344214f55df0aa364bd3 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width