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

Improve the Issue Templates #2375

Merged
merged 5 commits into from
Apr 18, 2023

Conversation

sadmuphin
Copy link
Contributor

Background

Improves the Issue templates to include more helpful/necessary data. Also adds a call to action to join the discord to discuss their issues. Hopefully improves the situation with the issues backlog.

Changes

  • Added operating system selector
  • Made GPT-3 or GPT-4 a dropdown
  • Added logs section

If anyone has any changes or recommendations please comment here.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

Swiftyos
Swiftyos previously approved these changes Apr 18, 2023
@Pwuts Pwuts self-requested a review April 18, 2023 13:22
Copy link
Member

@Pwuts Pwuts left a comment

Choose a reason for hiding this comment

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

Nice, we really need this :)

A few suggestions for further improvement:

.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/1.bug.yml Outdated Show resolved Hide resolved
Comment on lines 11 to 13
- type: checkboxes
attributes:
label: ⚠️ Search for existing issues first ⚠️
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- type: checkboxes
attributes:
label: ⚠️ Search for existing issues first ⚠️
- type: markdown
attributes:
value: '### ⚠️ Search for existing issues first ⚠️'
- type: checkboxes
attributes:

Copy link
Member

Choose a reason for hiding this comment

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

Could you still add this? Right now the preview-popup for all issues is "⚠️ Search for existing issues f..." because that label/header is included in the submitted issue. Moving that header to a markdown item fixes that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wait I though I changed it. Guess it didn't get pushed. One sec

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is label a required property? @Pwuts

Copy link
Member

Choose a reason for hiding this comment

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

@sadmuphin sadmuphin changed the base branch from master to richbeales-patch-2 April 18, 2023 15:03
@sadmuphin sadmuphin changed the base branch from richbeales-patch-2 to master April 18, 2023 15:03
@sadmuphin
Copy link
Contributor Author

Uh ignore the base branch change 😅 @Pwuts everything looks good to you now? Anything else I should add/remove?

Also should I make any changes to the feature template or leave it as is?

@sadmuphin sadmuphin requested a review from Pwuts April 18, 2023 15:07
@p-i- p-i- merged commit 8d9505c into Significant-Gravitas:master Apr 18, 2023
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants