Skip to content

Conversation

axe312ger
Copy link
Contributor

No description provided.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Dec 5, 2024
@serhalp serhalp added type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change topic: source-contentful Related to Gatsby's integration with Contentful labels Dec 11, 2024
@serhalp
Copy link
Member

serhalp commented Dec 11, 2024

Thanks @axe312ger!

It looks like the tests are failing, e.g.

/home/circleci/project/packages/gatsby-source-contentful/src/__tests__/fetch-network-errors.js

  ● fetch-retry › request should fail after to many retries

    expect(received).toEqual(expected) // deep equality

    Expected: StringContaining "Fetching contentful data failed: ERR_BAD_RESPONSE 500 MockedContentfulError"
    Received: "Fetching contentful data failed: ERR_BAD_RESPONSE 500 500 MockedContentfulError Mocked message of Contentful error··
    ---
    Request ID: 123abc·
    Attempts: 3·
    ---

@axe312ger
Copy link
Contributor Author

Yes, thanks. Just on it! :)

@axe312ger
Copy link
Contributor Author

Had to do 772d895 and 8d93859 to make CI happy. Please extract, remove, ignore, merge as you like @Gatsby Team

axe312ger and others added 4 commits December 12, 2024 11:36
Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
* debug: adapter e2e tests

* Update netlify.mjs

* Update netlify.mjs

* Update netlify.mjs

* Update remote-file.cy.ts

* Update netlify.mjs
@pieh pieh merged commit 3ccd1d8 into master Dec 13, 2024
38 checks passed
@pieh pieh deleted the build/contentful-v11 branch December 13, 2024 09:57
pieh added a commit that referenced this pull request Dec 13, 2024
* feat: update to ESM compatible version of CTF library and support latest rich text types

* build: clean up lock file

* fix: ensure we pass the corrent data to variable declaration check

* build: redo yarn.lock based on current master

* fix: make sure tests pass with new ctf client

* build: upgrade node version of circle ci e2e tests to match minimum engine version

* build: dedupe yarn.lock

* build: ensure we don't have duplicate versions of @contentful/rich-text-types

* Update .circleci/config.yml

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>

* Update .circleci/config.yml

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>

* test: adjust adapter e2e tests (#39182)

* debug: adapter e2e tests

* Update netlify.mjs

* Update netlify.mjs

* Update netlify.mjs

* Update remote-file.cy.ts

* Update netlify.mjs

* Revert "fix: ensure we pass the corrent data to variable declaration check"

This reverts commit 772d895.

---------

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
(cherry picked from commit 3ccd1d8)
pieh pushed a commit that referenced this pull request Dec 13, 2024
* feat: update to ESM compatible version of CTF library and support latest rich text types

* build: clean up lock file

* fix: ensure we pass the corrent data to variable declaration check

* build: redo yarn.lock based on current master

* fix: make sure tests pass with new ctf client

* build: upgrade node version of circle ci e2e tests to match minimum engine version

* build: dedupe yarn.lock

* build: ensure we don't have duplicate versions of @contentful/rich-text-types

* Update .circleci/config.yml

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>

* Update .circleci/config.yml

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>

* test: adjust adapter e2e tests (#39182)

* debug: adapter e2e tests

* Update netlify.mjs

* Update netlify.mjs

* Update netlify.mjs

* Update remote-file.cy.ts

* Update netlify.mjs

* Revert "fix: ensure we pass the corrent data to variable declaration check"

This reverts commit 772d895.

---------

Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
(cherry picked from commit 3ccd1d8)

Co-authored-by: Benedikt Rötsch <axe312ger@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer topic: source-contentful Related to Gatsby's integration with Contentful type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants