Skip to content

Conversation

@MenKNas
Copy link
Contributor

@MenKNas MenKNas commented Aug 11, 2025

Description / Motivation

This PR restores the local API fallback when contextId is missing (when users want to use local connection first), ensuring correct Edge/local connection priority and updated error messages.

Testing Details

  • Unit Test Added
  • Manual Test/Other (Please elaborate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@MenKNas MenKNas self-assigned this Aug 11, 2025
@github-actions
Copy link

github-actions bot commented Aug 11, 2025

📦 Package Size and Test Coverage Report

Package Base Size PR Size Δ Change Base Coverage PR Coverage Δ Change
cli 38.13 KB 38.13 KB ✅ 0.00 KB 57.42% 57.42% 0.00%
core 370.77 KB 371.98 KB 🔺 +1.21 KB 93.75% 93.68% -0.07%
create-content-sdk-app 103.07 KB 103.07 KB ✅ 0.00 KB 96.22% 96.22% 0.00%
nextjs 310.96 KB 310.96 KB ✅ 0.00 KB 85.21% 85.21% 0.00%
react 179.38 KB 179.38 KB ✅ 0.00 KB 92.31% 92.31% 0.00%
Total 🔺 +1.21 KB

@MenKNas MenKNas requested a review from stasmaxymov August 11, 2025 10:30
@MenKNas MenKNas merged commit dc2fcfb into dev Aug 11, 2025
2 checks passed
@MenKNas MenKNas deleted the bugfix/JSS-7256-restore-local-connection-back-to-csdk branch August 11, 2025 12:42
MenKNas added a commit that referenced this pull request Aug 13, 2025
* [core] Restore back local connection (local.api creds)

* [core] Update CHANGELOG.md

* [core] Fix linting issue

* [core] Address PR comment

(cherry picked from commit dc2fcfb)
MenKNas added a commit that referenced this pull request Aug 13, 2025
* [core] Restore back local connection (local.api creds)

* [core] Update CHANGELOG.md

* [core] Fix linting issue

* [core] Address PR comment

(cherry picked from commit dc2fcfb)
illiakovalenko added a commit that referenced this pull request Aug 13, 2025
* [core] Restore back local connection (local.api creds) (#178)

* [core] Restore back local connection (local.api creds)

* [core] Update CHANGELOG.md

* [core] Fix linting issue

* [core] Address PR comment

(cherry picked from commit dc2fcfb)

* [create-content-sdk-app] Add extra fix for GQL Client error when only local api is provided (#180)

* [create-content-sdk-app] Refactor middleware to work for all scenarios

* [create-content-sdk-app] Minor formatting fix

* [create-content-sdk-app] Restore comments

(cherry picked from commit e6ebeba)

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Illia Kovalenko <23364749+illiakovalenko@users.noreply.github.com>

* Update CHANGELOG.md

---------

Co-authored-by: Illia Kovalenko <23364749+illiakovalenko@users.noreply.github.com>
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.

3 participants