Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: [] Add raw data param for partial unpublish #2529

Merged
merged 3 commits into from
Jan 28, 2025

Conversation

colomolo
Copy link
Contributor

@colomolo colomolo commented Jan 27, 2025

Summary

Adding optional data param for partial unpublish as we pass version in the header.

Description

Motivation and Context

Checklist (check all before merging)

  • Both unit and integration tests are passing
  • There are no breaking changes
  • Changes are reflected in the documentation

@colomolo colomolo marked this pull request as ready for review January 27, 2025 16:57
@colomolo colomolo requested a review from a team as a code owner January 27, 2025 16:57
@colomolo colomolo enabled auto-merge (squash) January 28, 2025 20:54
@colomolo colomolo merged commit 6a938fa into master Jan 28, 2025
7 checks passed
@colomolo colomolo deleted the partial-unpublish-params branch January 28, 2025 21:07
@contentful-automation
Copy link
Contributor

🎉 This PR is included in version 11.45.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

mgoudy91 added a commit that referenced this pull request Feb 4, 2025
* fix: clean dependencies, delete unused file, migrated overlooked tests from last merge

* refactor: build process now supports TS, ESM, CJS and browser

* build: upgrade to ESM version of contentful-sdk-core

* test: reduce bundle size limits thanks to our optimizations

* build: clean up depdency changes

* chore: typos

* feat: this SDK now also includes its version number in its bundles

* test: fix output integration tests

* build: npm dedupe

* build: latest rollup dependencies

* fix: use with keyword instead of assert to mark import as json

* build: update depndencies related to this branch

* build: make npm happy to resolve our dependencies without --force

* test: ensure new failing typescript lint rules throw warnings only - still should be resolved at some point

* test: fix import paths

* build: ensure json-patch is available for typescript in projects using this api client

* ci: update node version

* fix: adjust import path

* style: ensure prettier doesnt fail on our rollup config file

* style: format code to make prettier lint happy

* style: format code to make prettier lint happy #2

* ci: update circleci config to match our new setup

* ci: cleanup

* test: slightly increase bundle size limits

* build(deps-dev): bump the dev-dependencies group across 1 directory with 8 updates (#2531)

Bumps the dev-dependencies group with 8 updates in the / directory:

| Package | From | To |
| --- | --- | --- |
| [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) | `7.26.0` | `7.26.7` |
| [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) | `7.26.0` | `7.26.7` |
| [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) | `9.18.0` | `9.19.0` |
| [eslint](https://github.com/eslint/eslint) | `9.18.0` | `9.19.0` |
| [lint-staged](https://github.com/lint-staged/lint-staged) | `15.4.1` | `15.4.3` |
| [playwright](https://github.com/microsoft/playwright) | `1.49.1` | `1.50.0` |
| [typescript](https://github.com/microsoft/TypeScript) | `5.6.3` | `5.7.3` |
| [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint) | `8.21.0` | `8.22.0` |



Updates `@babel/core` from 7.26.0 to 7.26.7
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.26.7/packages/babel-core)

Updates `@babel/preset-env` from 7.26.0 to 7.26.7
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.26.7/packages/babel-preset-env)

Updates `@eslint/js` from 9.18.0 to 9.19.0
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/commits/v9.19.0/packages/js)

Updates `eslint` from 9.18.0 to 9.19.0
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](eslint/eslint@v9.18.0...v9.19.0)

Updates `lint-staged` from 15.4.1 to 15.4.3
- [Release notes](https://github.com/lint-staged/lint-staged/releases)
- [Changelog](https://github.com/lint-staged/lint-staged/blob/master/CHANGELOG.md)
- [Commits](lint-staged/lint-staged@v15.4.1...v15.4.3)

Updates `playwright` from 1.49.1 to 1.50.0
- [Release notes](https://github.com/microsoft/playwright/releases)
- [Commits](microsoft/playwright@v1.49.1...v1.50.0)

Updates `typescript` from 5.6.3 to 5.7.3
- [Release notes](https://github.com/microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.6.3...v5.7.3)

Updates `typescript-eslint` from 8.21.0 to 8.22.0
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.22.0/packages/typescript-eslint)

---
updated-dependencies:
- dependency-name: "@babel/core"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dev-dependencies
- dependency-name: "@babel/preset-env"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dev-dependencies
- dependency-name: "@eslint/js"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: lint-staged
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dev-dependencies
- dependency-name: playwright
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: typescript-eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: add function logs API [EXT-6001] (#2520)

* feat: add function logs

- Added function log entity and plain client
- Added unit tests for function log
- Added js docstrings

* fix: prettier 😄

* fix: Add query params to getManyFunctions & getManyFunctionsForEnv

* feat: Added Functions and FunctionLogs to waterfall client

- Functions: get, getMany, getManyForEnvironment
- FunctionLogs: get, getAll

* fix: FunctionLog accepts query params & verbiage change

getAll() => getMany

* fix: Adjusted variable names

* fix: attempted adding query params to get many function logs

* fix: Adjusted query params and doc strings

* fix: Update jsDoc strings

* Revert "fix: Update jsDoc strings"

This reverts commit 2e759d5.

* fix: Reapply 6a7203f and add unit tests

* fix: add raw data param for partial unpublish (#2529)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Benedikt Rötsch <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michael Pineiro <[email protected]>
Co-authored-by: Yuri Mazursky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants