Skip to content

Add/tre guides - #8

Merged
vpnu merged 10 commits into
mainfrom
add/tre-guides
Jul 27, 2026
Merged

Add/tre guides#8
vpnu merged 10 commits into
mainfrom
add/tre-guides

Conversation

@vpnu

@vpnu vpnu commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

📝 Documentation

PR Description

Adds:

  • Approve Projects & memberships
  • Review Outputs
    Any links to pages/explanations/references that don't exist yet or are still in review need to be added later. This does mention Add Keycloak User but we said we would refactor that, so for now it's just mentioned.

Comment thread app/connect/approvals/page.mdx Outdated

### Knowledge

- Understand how the TRE Agent synchronises with the Submission Layer.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe a link back to app/connect/connect-submission/page.mdx or a subsection of it?

Comment thread app/connect/approvals/page.mdx Outdated
- A TRE Administrator user. A Keycloak administrator must create a TRE administrator account with the `dare-tre-admin` role. See **Add Keycloak User**.
- A connected TRE Agent. See [Connect a TRE Agent to the Submission Layer](/connect/connect-submission).

## Synchronisation

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved it down in a callout where it was meant to be 👍

Comment thread app/page.mdx Outdated
Comment thread app/page.mdx Outdated

@AndrewThien AndrewThien left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls add the links to what's next section of the page connect-submission as well

vpnu and others added 4 commits July 27, 2026 14:33
Co-authored-by: Tri Thien Nguyen <119530400+AndrewThien@users.noreply.github.com>
Co-authored-by: Tri Thien Nguyen <119530400+AndrewThien@users.noreply.github.com>
@vpnu
vpnu merged commit 7e62ee1 into main Jul 27, 2026
1 check passed
@vpnu
vpnu deleted the add/tre-guides branch July 27, 2026 14:57
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.

2 participants