Skip to content

add documentation page for canvas manifest - #891

Draft
nmpsilva wants to merge 2 commits into
mainfrom
nuno/update-canvas-manifest-docs
Draft

add documentation page for canvas manifest#891
nmpsilva wants to merge 2 commits into
mainfrom
nuno/update-canvas-manifest-docs

Conversation

@nmpsilva

Copy link
Copy Markdown
Contributor

No description provided.

@aws-amplify-us-west-2

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-891.d298pum72820gn.amplifyapp.com

@marythought

Copy link
Copy Markdown
Contributor

❤️ 💟 😻 🚀 🤘

@marythought
marythought force-pushed the nuno/update-canvas-manifest-docs branch from 350b926 to 4336651 Compare October 21, 2025 23:25
**Optional Fields:**
- `meta` (object): Additional metadata
- `data_access` (object): Defines what data the protocol can read/write
- `event` (string): The event type to listen for

@marythought marythought Oct 21, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is correct, and also it listen to multiple events so I wonder how much effort it would be to change to an array?

"secrets": [
"OPENAI_API_KEY"
],
"tags": {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this right? should it be "tags": ["interventions_and_safety"] ?


#### `global`

**Description:** Application is available organization-wide, not tied to any specific patient context.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we expect an application in the global context to ALSO show in patient specific contexts? I would assume yes, but am hearing some reports of this not working

@marythought
marythought force-pushed the nuno/update-canvas-manifest-docs branch from 4336651 to 074d575 Compare October 29, 2025 15:12

### Screenshot Placeholders

_TODO: Add screenshots showing where each scope type appears in the Canvas UI_

@marythought marythought Oct 29, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just calling out TODO so this doesn't get merged until added (or remove the TODO and add screenshots in a follow up PR would also be fine by me)

@canvas-sisyphus

Copy link
Copy Markdown
Contributor

Rebased nuno/update-canvas-manifest-docs onto main (resolved conflicts locally) and force-pushed, preserving the branch's individual commits. Force-pushed rebased nuno/update-canvas-manifest-docs to f986763 (2 commits, 3 file changes)

@canvas-sisyphus
canvas-sisyphus Bot force-pushed the nuno/update-canvas-manifest-docs branch from 074d575 to f986763 Compare August 27, 2026 07:39
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.

3 participants