feat: entry.update supports releaseId [DX-210] #2728
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
michaelphamcf
changed the title
michaelphamcf
changed the title
michaelphamcf
force-pushed
the
feat/DX-210-entry-update-to-take-release-id-2
branch
from
51db2d8 to
412a4fb
Compare
whitelisab
approved these changes
Sep 2, 2025
Contributor
whitelisab
left a comment
whitelisab
left a comment
There was a problem hiding this comment.
Just one clarification question, otherwise approving!
lib/adapters/REST/endpoints/entry.ts
Outdated
| export const update: RestEndpoint<'Entry', 'update'> = <T extends KeyValueMap = KeyValueMap>( | ||
| http: AxiosInstance, | ||
| params: GetSpaceEnvironmentParams & { entryId: string }, | ||
| params: UpdateEntryParams, |
Contributor
There was a problem hiding this comment.
Do we need to support QueryParams here if releaseEntry.update has that as part of the Params type?
Contributor
Author
There was a problem hiding this comment.
Oh right, I had that mentally noted but forgot! Thanks for the reminder.
michaelphamcf
deleted the
feat/DX-210-entry-update-to-take-release-id-2
branch
Contributor
|
🎉 This PR is included in version 11.55.0-canary.6 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Contributor
|
🎉 This PR is included in version 11.58.0-canary.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Contributor
|
🎉 This PR is included in version 11.59.0-canary.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
whitelisab
added a commit
that referenced
this pull request
Sep 24, 2025
* feat: add releaseSchema defaults to plain client creation [DX-184] (#2664) * feat: add releaseSchema defaults to plain client creation * fix: rename releaseSchema to releaseSchemaVersion * fix: forgot to fix test * feat(default): release-id (#2666) * feat: add support for releaseSchemaVersion in create/query methods [DX-187] (#2665) * feat: add releaseSchema defaults to plain client creation * fix: rename releaseSchema to releaseSchemaVersion * fix: forgot to fix test * feat: add support for releaseSchemaVersion in create/query methods * chore: lint fixes * feat: add support for release.entry.* pattern * chore: lint fixes * chore: fix test description * feat: add support for release.entry.* pattern [DX-192] (#2667) * feat: add releaseSchema defaults to plain client creation * fix: rename releaseSchema to releaseSchemaVersion * fix: forgot to fix test * feat: add support for releaseSchemaVersion in create/query methods * chore: lint fixes * feat: add support for release.entry.* pattern * chore: lint fix * chore: removing duplicate type * feat: release/update endpoint reflects schemaVersion param [DX-188] (#2668) * feat: add releaseSchema defaults to plain client creation * fix: rename releaseSchema to releaseSchemaVersion * fix: forgot to fix test * feat: add support for releaseSchemaVersion in create/query methods * chore: lint fixes * feat: add support for release.entry.* pattern * chore: lint fixes * feat: add forTemplatedSpaces field for env template creation [DX-173] (#2663) * chore: GetReleaseParams now extends GetReleaseEnvironmentParams * feat: release/update enpoint updated to reflect Release.v2 * chore: removed unnecessary error handling from update endpoint * chore: default integration test implemented * chore: withoutDefault integration test implemented * feat: all integrations tests passing * fix: test:integration-watch action in package.json reverted to default * fix: lint error resolved * fix: prettier passing --------- Co-authored-by: Ely Lucas <[email protected]> Co-authored-by: Lisa White <[email protected]> * Dx 194 Add unit tests for release.entry.get (#2674) * add unit tests and mock for ReleaseEntry * update test to use it.each * Remove wrapper tests; move Rest endpoint tests to proper directory * fix: lint prettier * feat(releases): release.entry.getMany() (#2673) * DX-202 Update entry.get to pass in releaseId (#2679) * feat: entry.get takes releaseId from params to retrieve release entry * add tests * chore:prettier * chore:prettier again * chore: move entry w/ releaseId tests to existing entry integration tests file * chore: cleanup comment, remove QueryParam that was added unnecessarily * chore: cleanup code * feat: update entry.getMany to include releasev2 params [DX-203] (#2685) * chore: getMany first attempt * chore: new integrations test file added (from Michael) * chore: type fixes implemented * chore: test shell added to entry-integration.test.ts * chore: tests implemented with necessary unimplemented release metadata tests commented out * chore: prettier fix * fix: entry integration scope removed from test script * chore: prettier fix * fix: version logic cleaned up and test query modified * chore: prettier fix * feat: add release.entry.update method [DX-206] (#2692) * feat: add release.entry.update method * feat: add entry.release.patch method (#2695) * fix: delete entry2 as part of entry integration test cleanup [DX-258] (#2700) * chore: add canary build to release/releasev2 branch [DX-212] (#2696) * chore: add canary build to release/releasev2 branch * chore: update readme with info on current experimental features * chore: creating canary branch for canary prerelease builds * chore: add canary branch to release process in ci config * feat: add release.entry.createWithId method [DX-61] (#2724) * feat: add release.entry.createWithId method [DX-61] * fix: update types for release entry params * test: debug integration tests * test: update unit tests and remove changes to integration tests * feat: entry.patch handle releaseId [DX-209] (#2725) * feat: entry.patch w/ releaseId alias for release.entry.patch * chore: add unit tests * chore: run prettier * chore: add integration tests for entry.patch with releaseId * chore: run lint * chore: cleanup code * chore: more cleanup * chore(test): remove redundant assertion * chore: cleanup params, console logs * chore: handle undefined headers for typescript * chore: use typecasting to pass param * feat: update entry.createWithId to accept releaseId [DX-208] (#2727) * feat: entry.update supports releaseId [DX-210] (#2728) * feat: entry.update supports releaseId * chore: add unit tests for entry.update * chore: use typecasting to pass param * test: add integration tests for entry.update * chore: keep QueryParams consistent between entry.update and release.entry.update * feat: add release.entry.create method (#2730) * feat: entry.create supports releaseId [DX-207] (#2732) * feat: add releaseId support to entry.create * chore: add QueryParams to be consistent with release.entry.create * fix: remove startDate from releasev2 payload [DX-380] (#2739) * feat: remove release[lte] endpoint [TOL-3408] (#2747) * feat: remove release[lte] endpoint * fix: delete more old tests, fix return types * fix(types): add release to entitymetasysprops [] (#2749) * feat: add support for release asset endpoints [TOL-3357] (#2750) * fix(types): add release to entitymetasysprops [] * feat(release-assets): add support for assets inside of a release [TOL-3357] * fix(types): export ReleasePaylodV2 [] (#2754) * fix: update types, add tests, and cleanup [DX-397] (#2755) * chore: update experimental features in readme --------- Co-authored-by: Ely Lucas <[email protected]> Co-authored-by: ethan ozelius <[email protected]> Co-authored-by: Tyler Pina <[email protected]> Co-authored-by: Michael Pham <[email protected]> Co-authored-by: MayaGillilan <[email protected]> Co-authored-by: Chris Helgert <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
JIRA: DX-210
entry.updatetakesreleaseIdand delegates to release entry update endpoint.Description
releaseIdto entry.update paramsreleaseId, callrelease.entry.updateendpointMotivation and Context
Checklist (check all before merging)
When adding a new method:
./lib/export-types.ts