-
Notifications
You must be signed in to change notification settings - Fork 202
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
Update issues link on the about page #758
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't solve the problem but it's much better for the user than seeing a GitHub login screen without explanation. LGTM!
We do also suggest 'sending an email' after the suggestion to create a GitHub issue. Email should be accessible and low-tech enough for most people, I think. |
Playwright failure test results It looks like some of the Playwright tests failed. You can download the Playwright trace https://github.com/WordPress/openverse/actions/runs/4244082120 Read more about how to use this artifact here: https://github.com/WordPress/openverse/blob/main/frontend/test/playwright/README.md#debugging |
Playwright failure test results It looks like some of the Playwright tests failed. You can download the Playwright trace https://github.com/WordPress/openverse/actions/runs/4244082120 Read more about how to use this artifact here: https://github.com/WordPress/openverse/blob/main/frontend/test/playwright/README.md#debugging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do also suggest 'sending an email' after the suggestion to create a GitHub issue. Email should be accessible and low-tech enough for most people, I think.
Yeah, thinking again, that seems sufficient for people not knowledgeable of GitHub and find the template for New Provider issues more valuable than linking to the GitHub issues page as well 😅 Perhaps we don't need to change anything here!
Fixes
Fixes #567 by @krysal
Description
Updates the "issue" link on the about page to point to the issues landing page. This is much clearer for users who do not already have a GitHub account or who aren't logged in.
Testing Instructions
Go to the /about page locally and click the "issue" link. Observe you are taken to the catalog issue list.
Checklist
main
) ora parent feature branch.
errors.
Developer Certificate of Origin
Developer Certificate of Origin