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

Issue templates/playground online #1263

Open
wants to merge 10 commits into
base: trunk
Choose a base branch
from

Conversation

flexseth
Copy link
Collaborator

@flexseth flexseth commented Apr 17, 2024

Requirements

Enable issue templates on GitHub

What is this PR doing?

Introduces an issue template that is fairly complete for playground.wordpress.net
AKA "Playground online"

What problem is it solving?

Provides guidance when creating a new issue for playground.wordpress.net

How is the problem addressed?

Created an issue template

Testing instructions

@adamziel
Copy link
Collaborator

Thanks @flexseth! This is a very thorough template, to the point where it may be scary to a new person coming in to report their problem. I, for once, would probably give up before reading the entire thing. How short can we make this, while still retaining the most important points? Could it be, e.g., a short paragraph of text and a checklist?

one of a couple updates to simplify
This may be a little too simple. Let's see how the users interact with adding their blueprints to tickets and see if it needs to be improved over time
Copy link
Collaborator Author

@flexseth flexseth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adamziel - check it out now, still too complicated?

Is there a way to test as if you were going to create a new issue, without merging it?

It for sure looks different in the New issue UI I feel like...

@adamziel
Copy link
Collaborator

adamziel commented Apr 19, 2024

Looking better, thank you! A few notes:

  • Some sections feature HTML comments, some feature plain text, others feature both. Let's stick to one style. Same for headers, sometimes it's ##, sometimes it's **.
  • The example list illustrating steps to reproduce isn't very clear, let's use a few more words like "1. I clicked ..., 2. I saw ..., 3. I expected ... instead".
  • In markdown you typically add a newline below headers. Two newlines typically mean an additional <br> tag. Let's make sure each header has a single newline above and below.

@adamziel
Copy link
Collaborator

Also there's some duplication between these:


## Journey to reproduce issue
1. Go to '...'
2. Expected
3. What happened
4. See errors - if applicable


## Expected behavior
A clear and concise description of what you expected to happen.

The Error reporting feels unnecessary for now, not many errors hinged on a specific WP version.

@flexseth
Copy link
Collaborator Author

Looking better, thank you! A few notes:

  • Some sections feature HTML comments, some feature plain text, others feature both. Let's stick to one style. Same for headers, sometimes it's ##, sometimes it's **.

HTML comment tags - they won't show up on the issue front-end

I can take a look at the other part. Think I adapted it, thank you for the review

  • The example list illustrating steps to reproduce isn't very clear, let's use a few more words like "1. I clicked ..., 2. I saw ..., 3. I expected ... instead".

Ok, didn't know what to put there. Maybe even just removing the steps all together and let them write what they are experiencing? Leads to more triage, but also helps resolve more issues which could become some sort of knowledge base

  • In markdown you typically add a newline below headers. Two newlines typically mean an additional <br> tag. Let's make sure each header has a single newline above and below.

Heard

@adamziel
Copy link
Collaborator

adamziel commented Apr 23, 2024

Ok, didn't know what to put there. Maybe even just removing the steps all together and let them write what they are experiencing? Leads to more triage, but also helps resolve more issues which could become some sort of knowledge base

It could be something to this effect:

## What happened? (step by step)

1. I went to https://playground.wordpress.net/ and clicked ...
2. I then clicked ...
3. And ... happened

## What did you expect to happen instead?

<!-- Fill in the blank :) -->

## Blueprints, screenshots, additional context

<!-- If you used a Blueprint, have a screenshot to share, or any other relevant thoughts, put them here -->

## Console errors

<!-- Please include the output from your web browser console. If you're not sure how to do that, see https://support.google.com/docs/thread/1873663/collecting-console-logs-chrome-browser-only   -->

@flexseth flexseth requested a review from adamziel April 25, 2024 03:39
@ironnysh
Copy link
Collaborator

ironnysh commented May 5, 2024

Suggestion: add a GitHub workflow to auto-label issues when submitting (see example on WordPress Learn)

@flexseth
Copy link
Collaborator Author

flexseth commented May 5, 2024

Suggestion: add a GitHub workflow to auto-label issues when submitting (see example on WordPress Learn)

Not a bad idea at all! There have been quite a few more labels added to this repo

h/t to @adamziel for the implementation!

What would be really cool is if we could set up a GitHub Action to search for and label certain issues, based on search criteria found in the subject and issue description.

Adding to internal ideas so it won't get lost

@@ -0,0 +1,36 @@
---
name: playground.wordpress.net - Issues
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name: playground.wordpress.net - Issues
name: Bug Report – playground.wordpress.net

---
name: playground.wordpress.net - Issues
about: Something did not work as expected in the online version of Playground?
title: 'Playground Online: Report an issue with playground.wordpress.net'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title: 'Playground Online: Report an issue with playground.wordpress.net'
title: 'Bug Report playground.wordpress.net'

Btw how is name different than title? Also, shouldn't this be a .yml file living in .github/ISSUE_TEMPLATE? Or will it work as a .md file living in .github?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants