-
Notifications
You must be signed in to change notification settings - Fork 59.8k
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
Fixes the description about id
key in Issue Forms.
#9045
Conversation
Fixes #9004 The `id` key is the top level key and has been misplaced in the `attributes` key. Luckily the examples do not have any such error,
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.
Nice one @thispsj !
@thispsj Thanks so much for opening a PR! I got this triaged for review ⚡ |
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.
This is fantastic, thank you @thispsj! ✨
I've made one little tweak which I'll apply in a second before merging and getting these changes deployed.
Thanks again for taking the time to contribute to our documentation!
...ing-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-githubs-form-schema.md
Outdated
Show resolved
Hide resolved
@thispsj 👋 Would you mind either enabling the checkbox to allow maintainer edits (so I can commit the suggested change above) or committing it yourself? Then I can get this merged in. Thank you! |
ece3a8f
@stevecat done. |
...ing-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-githubs-form-schema.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Steve Guntrip <12534592+stevecat@users.noreply.github.com>
@stevecat you can enable automerge now I guess |
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.
@thispsj Perfect, thanks so much for making those changes! I'll get this merged now ✨
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
Why:
Closes #9004
What's being changed:
The
id
key is used in issue forms with every type (excluding markdown). But the docs mention that theattributes
key should contain it which is wrong. Theid
key is a top-level key in each type.Check off the following: