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

Test new PR create views #5100

Closed
3 tasks done
alexr00 opened this issue Jul 24, 2023 · 3 comments
Closed
3 tasks done

Test new PR create views #5100

alexr00 opened this issue Jul 24, 2023 · 3 comments

Comments

@alexr00
Copy link
Member

alexr00 commented Jul 24, 2023

Refs: #4403

Complexity: 5

Authors: @alexr00, @hbons

Create Issue


We have update the Create PR webview and the Files changed views. Please do exploratory testing on the "Create" views.

Setup:

  • Install the pre-release build of the extension
  • The new views are enabled by default already in the pre-release version
  • Open a folder with a GitHub repository that you don't mind create PRs and pushing branches in
  • To open the "Create" views, use the "Create Pull Request" command or the "Commit & Create Pull Request" button in the SCM view.

Specific features to test:

  • Setting assignees, reviewers, labels, and a milestone from the create view
  • After setting assignees/reviewers/labels/milstones, confirm everything in the webview is tabbable
  • We try to guess your "into" branch better: If the branch you branched off from has an remote, then that will be your into branch. Otherwise, it will be the default branch for the repo
  • In the branch selector, confirm ‘Change Repository…’ lists the repositories in the network
  • The "Create" button remembers your last choice
  • The "Files Changed" view shows changes even when your branch hasn't been pushed yet
  • The "Files Changed" and "Commits" view update when you change branches or push your "from" branch.
  • The controls in the webview are disabled when you open the view as we load the data for the view
  • The controls in the webview are disabled after you click the "Create" button and while the PR is being created
@aiday-mar
Copy link
Contributor

aiday-mar commented Jul 25, 2023

HI @alexr00, I am currently testing the work. While testing, I had the following thoughts:

  1. It seems as though it is currently not possible to create labels (perhaps I have missed the button to do so). However it is possible to create milestones from the view. Do you think it would be useful to be able to create labels too?
  2. After I set the assignee, the reviewer and the label I notice the following:

Screenshot 2023-07-25 at 09 44 05

There appears to be little space between the reviewer/assignee name and their circular picture. Perhaps a little padding could make it better.
3. Concerning the Change repository... button, I was wondering what repositories exactly I should expect to see?Currently the only repository shown is the one I am currently on. I was expecting to see the other repositories I host on my account.

Screenshot 2023-07-25 at 09 47 27

I like the new interface!

@aiday-mar aiday-mar removed their assignment Jul 25, 2023
@alexr00
Copy link
Member Author

alexr00 commented Jul 25, 2023

It seems as though it is currently not possible to create labels (perhaps I have not missed the button to do so). However it is possible to create milestones from the view. Do you think it would be useful to be able to create labels too?

Seems like a reasonable feature rerquest!

There appears to be little space between the reviewer/assignee name and their circular picture. Perhaps a little padding could make it better.

@hbons already has a fix ready and is waiting on me: #5101

Concerning the Change repository... button, I was wondering what repositories exactly I should expect to see?Currently the only repository shown is the one I am currently on. I was expecting to see the other repositories I host on my account

You will only see repos that you have as git remotes.

@aiday-mar
Copy link
Contributor

aiday-mar commented Jul 25, 2023

Hi @alexr00, I see better now thanks!

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

No branches or pull requests

4 participants