Skip to content

Report issue with page option #2138

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

Merged
merged 8 commits into from
May 20, 2025
Merged

Report issue with page option #2138

merged 8 commits into from
May 20, 2025

Conversation

Leinnan
Copy link
Contributor

@Leinnan Leinnan commented May 17, 2025

Fixes #2135
Adds report issue button alongside edit page.
Links open new issue page on github that is using issue template that is also added.

@rparrett
Copy link
Contributor

I could see the generated issue titles being a bit confusing. Maybe something like

Issue%20on%20{{ section_or_page.title | urlencode | safe }}%20page

?

Co-authored-by: Rob Parrett <[email protected]>
@Leinnan
Copy link
Contributor Author

Leinnan commented May 17, 2025

I could see the generated issue titles being a bit confusing. Maybe something like


Issue%20on%20{{ section_or_page.title | urlencode | safe }}%20page

?

I like it, added 👌

@Leinnan Leinnan requested a review from rparrett May 17, 2025 19:13
@rparrett
Copy link
Contributor

I like it, added 👌

Not seeing that change on my end

@Leinnan
Copy link
Contributor Author

Leinnan commented May 19, 2025

@rparrett I was thinking you were talking mostly about urlencode. Updated

@Leinnan Leinnan requested a review from rparrett May 20, 2025 13:21
@rparrett rparrett added S-Ready-For-Final-Review Ready for a maintainer to consider for merging C-Feature A new feature, making something new possible A-Cross-Cutting Impacts multiple parts of the website labels May 20, 2025
@mockersf mockersf added this pull request to the merge queue May 20, 2025
Merged via the queue into bevyengine:main with commit 6852a26 May 20, 2025
10 checks passed
@Leinnan Leinnan deleted the report-docs branch May 21, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Cross-Cutting Impacts multiple parts of the website C-Feature A new feature, making something new possible S-Ready-For-Final-Review Ready for a maintainer to consider for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a "Report a Problem with this page" button
3 participants