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

Update to provide link to form to suggest feedback. #209

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ejh243
Copy link

@ejh243 ejh243 commented Dec 17, 2024

Added link to suggestions form. @berrli might need to check/test this is in the right format.

Verified

This commit was signed with the committer’s verified signature.
ggerganov Georgi Gerganov
@ejh243 ejh243 linked an issue Dec 17, 2024 that may be closed by this pull request

Verified

This commit was signed with the committer’s verified signature.
ggerganov Georgi Gerganov
…n .html
@berrli
Copy link
Collaborator

berrli commented Dec 17, 2024

Added link to suggestions form. @berrli might need to check/test this is in the right format.

Ive made a commit to the PR to modify the link to the suggestion form. The links within a JupyterBook need to point to the .ipynb file rather than the .html (as the GitHub Action generates the .html files). Opened an Issue to improve the guidance as i believe i have missed including details about this, #210

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.

Issue on page /contributing/roles.html
2 participants