Skip to content

Minor tweaks to entirePlanX resolvers to support updated REST API - #801

Open
briri wants to merge 1 commit into
developmentfrom
feature/update-delete-api-support
Open

Minor tweaks to entirePlanX resolvers to support updated REST API#801
briri wants to merge 1 commit into
developmentfrom
feature/update-delete-api-support

Conversation

@briri

@briri briri commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Part of #261

  • Added versionedTemplate schema and resolver so we can fetch a versioned template by its id.
  • Switched entirePlan schema, resolvers and service to use versionedXId instead of xId (e.g. use versionedTemplates instead of Templates)
  • Updated entirePlan service to use the default MemberRole when none is provided

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

REST API calls are working with switch to call entirePlan resolvers. I've done some limited testing, I'm sure there will be bugs that surface as we begin using and testing. I will

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I updated the CHANGELOG.md and added documentation if necessary
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@briri
briri requested a review from jupiter007 July 30, 2026 21:23
@briri
briri marked this pull request as ready for review July 30, 2026 21:31

@jupiter007 jupiter007 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.

Looks good. I didn't see any updates to the CHANGELOG.md file though

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