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

github: Configure templates for filing issues. #26000

Merged
merged 2 commits into from
Jun 13, 2023

Conversation

alya
Copy link
Contributor

@alya alya commented Jun 12, 2023

Follow-up to #25998, pushed as a separate PR so that the original one can pass CI.

This PR creates templates for filing issues. The templates are intentionally quite light-weight. Note that I'm specifically not using forms for creating issues, as the UI for filling out such a form does not include GitHub's helpful formatting buttons and preview mode.

More importantly, users will be presented with the following options when creating an issue:

Screen Shot 2023-06-12 at 3 27 48 PM

A major goal is to guide users towards starting a CZO conversation when that's more appropriate than filing a GitHub issue.

Note that the config makes it possible to create a blank issue, which should be handy for:

  • Issues filed by maintainers
  • Issues for tracking follow-ups from merged PRs
  • Probably some other situations

Because the blank issue option is easy to miss, it should probably be documented somewhere, but I'm not sure where. We can perhaps start with a note on CZO.

Relevant CZO threads:

Please review carefully, as my testing involved copy-pasting config files around to try things out.

@alya alya added the priority integration review integration reviews requesting prompt attention label Jun 12, 2023
@zulipbot zulipbot added size: L and removed size: XL labels Jun 12, 2023
@timabbott timabbott added the backport candidate Items we should consider backporting to the bug fix release series. label Jun 12, 2023
@timabbott timabbott force-pushed the reporting-bugs-continued branch 3 times, most recently from 3622754 to 21b4db2 Compare June 12, 2023 23:13
This PR creates templates for filing issues. The templates are
intentionally quite light-weight. Note that I'm specifically not using
forms for creating issues, as the UI for filling out such a form does
not include GitHub's helpful formatting buttons and preview mode.

Follow-up to zulip#25998, pushed as a separate PR so that the original one can pass CI.

This PR creates templates for filing issues. The templates are
intentionally quite light-weight. Note that I'm specifically not using
forms for creating issues, as the UI for filling out such a form does
not include GitHub's helpful formatting buttons and preview mode.

A major goal is to guide users towards starting a CZO conversation
when that's more appropriate than filing a GitHub issue.

Note that the config makes it possible to create a blank issue, which
should be handy for:

* Issues filed by maintainers
* Issues for tracking follow-ups from merged PRs
* Probably some other situations

Because the blank issue option is easy to miss, it should probably be
documented somewhere, but I'm not sure where. We can perhaps start
with a note on CZO.

Relevant CZO threads:

https://chat.zulip.org/#narrow/stream/137-feedback/topic/issues.20link.20in.20description/near/1561110)
https://chat.zulip.org/#narrow/stream/2-general/topic/bug.20report.20management/near/1589141

Also provide external documentation links for situations where
filing an issue is not the best approach.
@timabbott
Copy link
Sponsor Member

With the tweaks we did just now, this looks good to me. Merging, though of course because issue templates don't seem to have a preview mode :), we may need to make some further changes after testing it.

@timabbott timabbott enabled auto-merge (rebase) June 12, 2023 23:25
@timabbott timabbott merged commit 0adcc2a into zulip:main Jun 13, 2023
8 checks passed
@alexmv
Copy link
Collaborator

alexmv commented Jun 13, 2023

Backported as 67157ec / f4f8d09.

@alexmv alexmv removed the backport candidate Items we should consider backporting to the bug fix release series. label Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority integration review integration reviews requesting prompt attention size: L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants