Skip to content

Polish PR template picker - #151415

Merged
João Moreno (joaomoreno) merged 6 commits into
microsoft:mainfrom
babakks:polish-pr-template-picker
Jun 10, 2022
Merged

Polish PR template picker#151415
João Moreno (joaomoreno) merged 6 commits into
microsoft:mainfrom
babakks:polish-pr-template-picker

Conversation

@babakks

@babakks Babak K. Shandiz (babakks) commented Jun 7, 2022

Copy link
Copy Markdown
Contributor

This PR fixes #148323

Fixes include:

  • Relative paths for templates.
  • Set ignoreFocusOut: true.

João Moreno (@joaomoreno) Regarding things didn't work for Joyce Er (@joyceerhl) more than once, I'm not sure what we can do at the this step (i.e., showing the picker). I think it should be related to the previous step (i.e., handling the fork/push events). What's your take on this?

Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
@babakks

Copy link
Copy Markdown
Contributor Author

João Moreno (@joaomoreno) Could you please review this?

@joaomoreno

João Moreno (joaomoreno) commented Jun 10, 2022

Copy link
Copy Markdown
Contributor

João Moreno (@joaomoreno) Regarding things didn't work for Joyce Er (@joyceerhl) more than once, I'm not sure what we can do at the this step (i.e., showing the picker). I think it should be related to the previous step (i.e., handling the fork/push events). What's your take on this?

That's been fixed: #148326

Thanks for the PR!

@joaomoreno João Moreno (joaomoreno) added this to the June 2022 milestone Jun 10, 2022
@joaomoreno
João Moreno (joaomoreno) enabled auto-merge (squash) June 10, 2022 12:46
@joaomoreno
João Moreno (joaomoreno) merged commit 7667436 into microsoft:main Jun 10, 2022
@babakks
Babak K. Shandiz (babakks) deleted the polish-pr-template-picker branch June 10, 2022 20:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Github: PR template picker needs polish

3 participants