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.

Sign up for GitHub

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

Jump to bottom

Edit question template to direct users to GH Discussions #21393

Closed
wants to merge 1 commit into from
Closed

Edit question template to direct users to GH Discussions #21393

wants to merge 1 commit into from

Conversation

t8m
Copy link
Member

@t8m t8m commented Jul 7, 2023

No description provided.

@t8m t8m added branch: master Merge to master branch approval: review pending This pull request needs review by a committer approval: otc review pending triaged: documentation The issue/pr deals with documentation (errors) tests: exempted The PR is exempt from requirements for testing labels Jul 7, 2023
mattcaswell
mattcaswell reviewed Jul 7, 2023
View reviewed changes
.github/ISSUE_TEMPLATE/question.md Outdated
@@ -4,53 +4,7 @@ labels: 'issue: question'
about: Ask a question about OpenSSL
Copy link
Member

Choose a reason for hiding this comment

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

How about changing the "about" text too? You see this text on the "new issue" page before even clicking on the button to create a new question.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, good point

Copy link
Contributor

@mspncp mspncp Jul 10, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

I think the New Issues hint would be even more prominent and effective, if you would add a link to it:

Please use the Q&A category in GitHub Discussions to ask general questions about using OpenSSL.

That's possible, as the vulnerability hint demonstrates:

grafik

@t8m t8m force-pushed the issue-template-discussions branch from faa9a28 to a8a6881 Compare July 7, 2023 13:15
@t8m t8m force-pushed the issue-template-discussions branch from a8a6881 to d518e99 Compare July 7, 2023 13:16
@mspncp mspncp added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer approval: otc review pending labels Jul 7, 2023
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Jul 8, 2023
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@paulidale
Copy link
Contributor

Merged!

@paulidale paulidale closed this Jul 9, 2023
openssl-machine pushed a commit that referenced this pull request Jul 9, 2023
@t8m @paulidale
Edit question template to direct users to GH Discussions
bd38c6b 
Reviewed-by: Matthias St. Pierre <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from #21393)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mspncp mspncp mspncp approved these changes

@mattcaswell mattcaswell mattcaswell approved these changes

@paulidale paulidale paulidale approved these changes

Assignees
No one assigned
Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch tests: exempted The PR is exempt from requirements for testing triaged: documentation The issue/pr deals with documentation (errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@t8m @openssl-machine @paulidale @mattcaswell @mspncp