Add confirmation to issue reporter - #1749
Open
Eduardo Villalpando Mello (edvilme) wants to merge 4 commits into
Open
Add confirmation to issue reporter#1749Eduardo Villalpando Mello (edvilme) wants to merge 4 commits into
Eduardo Villalpando Mello (edvilme) wants to merge 4 commits into
Conversation
Eduardo Villalpando Mello (edvilme)
requested review from
Stella Huang (StellaHuang95) and
Eleanor Boyd (eleanorjboyd)
and removed request for
Eleanor Boyd (eleanorjboyd)
August 31, 2026 17:15
Contributor
|
Can you add a screenshot to show what the ui or user flow looks like now? |
Contributor
|
I like this additional step, but would like to hear if others have comments on the wordings. My thought is just that users may not be aware of this repo or don't know what it is, use maybe use some plain wordings like this will create an issue to help us investigate the issue you have with python environments or something? |
Stella Huang (StellaHuang95)
previously approved these changes
Aug 31, 2026
Member
|
yeah I like stella's idea here on the wording. There was also maybe a way you could open the issue as a draft on github instead of filing it right away? maybe check the api for the issue filing to see if thats possible |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added confirmation step to issue reporter to make intent clear to users and avoid accidental issue creation
This pull request makes improvements to user interaction and code organization in the
src/extension.tsfile. The main changes include adding a confirmation dialog before opening the issue reporter and making a small adjustment to the import order for better code clarity.User Experience Improvements:
Code Organization:
copyPathToClipboardfor improved consistency and readability in the import section.