Skip to content

feat: add Fragment getMany and get methods [DX-927]#3000

Merged
Michael Pham (michaelphamcf) merged 3 commits intofeat/exofrom
feat/exo-fragment-get
Apr 28, 2026
Merged

feat: add Fragment getMany and get methods [DX-927]#3000
Michael Pham (michaelphamcf) merged 3 commits intofeat/exofrom
feat/exo-fragment-get

Conversation

@michaelphamcf
Copy link
Copy Markdown
Contributor

Summary

  • Add getMany and get REST adapter endpoints for Fragment (GET /fragments, GET /fragments/:id)
  • Create FragmentPlainClientAPI interface with all method signatures (getMany, get, create, update, delete, publish, unpublish)
  • Wire Fragment MRActions block, MakeRequest overloads, and plain client registration
  • Unit tests for getMany (empty query, pagination, order filter) and get

Test plan

  • tsc --noEmit passes
  • npx vitest --project unit --run test/unit/adapters/REST/endpoints/fragment.test.ts — 4 tests pass
  • create/update/delete/publish/unpublish REST implementations follow in DX-928 and DX-929

Stacked PR — targets feat/exo-fragment-types (DX-926). Will auto-retarget to feat/exo when that merges.

🤖 Generated with Claude Code

@michaelphamcf Michael Pham (michaelphamcf) requested a review from a team as a code owner April 27, 2026 23:21
@bito-code-review
Copy link
Copy Markdown

bito-code-review Bot commented Apr 27, 2026

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at michael.pearce@contentful.com.

Base automatically changed from feat/exo-fragment-types to feat/exo April 28, 2026 20:38
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
MRActions already defined publish and unpublish for Fragment, and the
plain client wires both via wrap() — but the MRInternal overload block
was missing them, leaving the overload resolution incomplete.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@michaelphamcf Michael Pham (michaelphamcf) merged commit e9b189d into feat/exo Apr 28, 2026
19 of 21 checks passed
@michaelphamcf Michael Pham (michaelphamcf) deleted the feat/exo-fragment-get branch April 28, 2026 21:27
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