Skip to content

Document the PlanDefinition route required for form packages - #5

Open
alex-serednev wants to merge 1 commit into
mainfrom
docs/smart-plandefinition-access
Open

alex-serednev wants to merge 1 commit into
mainfrom
docs/smart-plandefinition-access

Conversation

@alex-serednev

@alex-serednev alex-serednev commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

What

Adds the PlanDefinition route to the SMART access policy on the EHR integration via SMART on FHIR page, plus a new Form packages access section covering why the route is needed.

Why

Form packages are stored as PlanDefinition resources, and the Forms page reads them alongside questionnaires. A SMART access policy that does not include that route leaves the forms list unable to display correctly, so the route belongs in the documented policy.

The policy generated by the SMART setup in the Formbox UI does not include it either, so the new section explains how to add it there, and notes that covering it in the generated policy can be done by request.

Changes

  • docs/aidbox-ui-builder-alpha/smart-app-integration.md
    • ### 4. Access policy — added the PlanDefinition pattern to the $one-of list
    • new ### Form packages access section, placed after the setup steps and before ### Sandbox note

Validation

bun lint — 12 checks passed, 5 warnings, all pre-existing and in other files. Output is identical before and after this change.

🤖 Generated with Claude Code

@alex-serednev
alex-serednev requested a review from qdzo September 19, 2026 06:24
The Forms page reads form packages alongside questionnaires, so the SMART
access policy needs to allow the PlanDefinition route.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@alex-serednev
alex-serednev force-pushed the docs/smart-plandefinition-access branch from ca3e30a to 000bf4a Compare September 19, 2026 06:34
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.

1 participant