Repository navigation
feat: add discussion forms and contact links - #53
Merged
Merged
Conversation
Add category forms for Q&A and Adoption on the organisation's discussion board, whose source repository is this one. Point the issue chooser's contact links to the Q&A category, Slack and the support guide instead of the deprecated contact page, and describe the forms in the README. GitHub's schema allows only http(s) URLs in contact links, so "Contact the maintainers" links the support guide (SUPPORT.md) and names the email address in its description. Closes #43 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewer's GuideAdds validated Q&A and Adoption discussion forms, updates issue contact links to direct users to discussions, Slack, maintainers, or security reporting, and documents the discussion-board configuration in the README. Flow diagram for structured discussion formsflowchart TD
Start["Start a discussion"] --> Category{"Choose category"}
Category --> Q["Q&A form"]
Category --> A["Adoption form"]
Q --> QFields["Repository, version, question, logs, prior-search confirmation"]
A --> AFields["Services, use, optional organisation and environment, adopter-list consent"]
QFields --> Post["Submit structured post"]
AFields --> Post
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
uniqueg
marked this pull request as ready for review
November 1, 2026 03:59
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path=".github/DISCUSSION_TEMPLATE/q-a.yml" line_range="2" />
<code_context>
+---
+body:
+ - type: markdown
+ attributes:
</code_context>
<issue_to_address>
**Discussion forms do not load**
When gitHub attempts to load either discussion category form, gitHub rejects each form because it has a `body` but no required top-level `title`, so users do not get the structured Q&A or Adoption form.
Add a top-level `title` to each discussion form, using the corresponding category title.
Also at `.github/DISCUSSION_TEMPLATE/adoption.yml:2`.
</issue_to_address>Linked issues
#43 feat: add discussion forms and contact links · 14 of 16 met · 1 to check by hand
| # | Requirement | Status | Notes |
|---|---|---|---|
| 1 | Add category forms named q-a.yml and adoption.yml | ✅ Met | Both forms are in .github/DISCUSSION_TEMPLATE/. |
| 2 | Q&A repository dropdown with the specified options | ✅ Met | |
| 3 | Ask for the version (release or commit) | ✅ Met | |
| 4 | Ask what the user wants to do and has tried | ✅ Met | |
| 5 | Collect logs or errors, render as shell, and remind users to remove secrets and tokens | ✅ Met | |
| 6 | Require confirmation that docs and existing issues/discussions were searched | ✅ Met | |
| 7 | Ask for the organisation, optionally | ✅ Met | |
| 8 | Multi-select services used, with the specified options | ✅ Met | |
| 9 | Ask for the environment, optionally | ✅ Met | |
| 10 | Ask what the adopter uses the services for | ✅ Met | |
| 11 | Include a checkbox granting permission to list the adopter | ✅ Met | |
| 12 | Link “Ask a question” to the Q&A category | ✅ Met | |
| 13 | Link “Chat with the community” to the specified Slack invite | ✅ Met | |
| 14 | Link “Contact the maintainers” to mailto:cloud-service@elixir-europe.org | 🟡 Partly met | The contact link goes to SUPPORT.md instead; the email address is included in its description. |
| 15 | Keep the vulnerability link pointing to the security policy | ✅ Met | |
| 16 | Verify the forms on the board by opening a new discussion in each category after merge | 👀 Check by hand (runtime) | After merge, open a new discussion in Q&A and Adoption on GitHub and confirm each form appears as expected. |
Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: .github/DISCUSSION_TEMPLATE/q-a.yml:2
There was a problem hiding this comment.
Linked issues
#43 feat: add discussion forms and contact links · 8 of 10 met · 1 to check by hand
| # | Requirement | Status | Notes |
|---|---|---|---|
| 1 | Q&A form: repository dropdown with the listed options | ✅ Met | The form includes all seven named projects and “other”. |
| 2 | Q&A form: version, question, and logs fields | ✅ Met | The question asks what the user wants to do and has tried; the logs field renders as shell and asks users to remove secrets and tokens. |
| 3 | Q&A form: require confirmation that docs, issues, and discussions were searched | ✅ Met | The required checkbox names all three. |
| 4 | Adoption form: optional organisation and environment, services multi-select, and use field | ✅ Met | The form includes the optional fields, the listed services as a multi-select, and a required use field. |
| 5 | Adoption form: checkbox allowing the adopter to be listed | ✅ Met | The checkbox says the adopter may be listed and gives elixir-on.cloud as an example. |
| 6 | Contact links: Ask a question to the Q&A category | ✅ Met | |
| 7 | Contact links: Chat with the community to the specified Slack invite | ✅ Met | |
| 8 | Contact maintainers to mailto:cloud-service@elixir-europe.org | 🟡 Partly met | The link goes to SUPPORT.md rather than the requested mailto URL; the email address is named in the link description, as the PR says GitHub's schema rejects mailto contact links. |
| 9 | Report a vulnerability to the security policy | ✅ Met | |
| 10 | Verify the forms on the board by starting a discussion in each category after merge | 👀 Check by hand (external system) | After merge, open a new discussion in both Q&A and Adoption on GitHub and confirm the corresponding form appears. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Structures posts on the org discussion board (source repository: this one)
and updates the issue chooser's contact links:
.github/DISCUSSION_TEMPLATE/q-a.yml: repository (dropdown: CloudRegistry, Cloud Registry UI, Cloud Components, FOCA, FOCA plugins, proTES,
TRS-Filer, other), version, question (what you want to do and what you
tried), logs (rendered as
shell, with a note to remove secrets andtokens), required checkbox that the docs, issues and discussions were
searched.
.github/DISCUSSION_TEMPLATE/adoption.yml: organisation (optional),services (multi-select, same options), environment (optional), use, and a
checkbox allowing us to list the adopter (e.g. on elixir-on.cloud).
.github/ISSUE_TEMPLATE/config.yml: "Ask a question" (Q&A category),"Chat with the community" (Slack invite), "Contact the maintainers",
"Report a vulnerability" (security policy, as before). The link to the
deprecated
elixir-cloud-aai.github.iocontact page is gone.Deviation from the issue: GitHub's schema for
config.ymlallows onlyhttp(s)URLs in contact links (check-github-issue-configrejectsmailto:cloud-service@elixir-europe.org). "Contact the maintainers"therefore links the support guide (
SUPPORT.md, #44 / #52, merged first)and names the email address in its description.
Closes #43
Light lane
Configuration and docs text only (YAML forms, contact links, README), no
logic: no reviewer subagents; I reviewed the diff myself.
How it was verified
pre-commit run --all-files: all hooks passed (including thecheck-github-issue-configschema check of the new contact links).pre-commit run --all-files --hook-stage manual gitleaks-all: passed.against GitHub's issue-forms schema (vendored by check-jsonschema), with
dummy values for the issue-only keys
nameanddescription, and checkedthat the top-level keys are only
title/labels/bodyand that eachbody has a non-markdown field: both passed.
forms on the board.
Declined review findings
title: GitHub documentstitleandlabelsas optional; onlybodywith a non-Markdown field is required.Open questions
None.
🤖 Generated with Claude Code