Skip to content

feat: add is_sso parameter to migration and content type handling fun…#1024

Merged
umeshmore45 merged 3 commits intodevfrom
feature/cmg-774
Apr 2, 2026
Merged

feat: add is_sso parameter to migration and content type handling fun…#1024
umeshmore45 merged 3 commits intodevfrom
feature/cmg-774

Conversation

@AishDani
Copy link
Copy Markdown
Contributor

@AishDani AishDani commented Apr 1, 2026

…ctions

umeshmore45
umeshmore45 previously approved these changes Apr 1, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the migration/content-type creation flow to propagate an is_sso flag (so downstream calls can choose SSO access tokens vs basic authtoken auth), and refines UI-side content type mapping resolution to prefer the latest local mapping state with a Redux fallback.

Changes:

  • UI: resolve mapped destination content type UID using local contentTypeMapped first, then Redux content_type_mapping, and update effect dependencies accordingly.
  • API: add is_sso propagation from migration.servicefieldAttachercontenTypeMakerexistingCtMapper token payload.
  • API: include is_sso in the synthetic request payloads used to call contentMapperService for existing CT/global-field lookups.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
ui/src/components/ContentMapper/index.tsx Updates mapping lookup logic and effect dependencies for selecting the mapped destination content type.
api/src/services/migration.service.ts Extracts and forwards is_sso from request token payload into the migration pipeline.
api/src/utils/field-attacher.utils.ts Accepts/forwards is_sso to content type creation.
api/src/utils/content-type-creator.utils.ts Threads is_sso into existing content type/global field retrieval calls via token_payload.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

umeshmore45
umeshmore45 previously approved these changes Apr 2, 2026
@AishDani AishDani dismissed stale reviews from umeshmore45 and vikrantraut-cstk via 416a6d5 April 2, 2026 09:01
@vikrantraut-cstk vikrantraut-cstk removed the request for review from vikrantraut April 2, 2026 10:41
@umeshmore45 umeshmore45 merged commit 8c4e12d into dev Apr 2, 2026
9 checks passed
@umeshmore45 umeshmore45 deleted the feature/cmg-774 branch April 2, 2026 10:42
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.

5 participants