Skip to content

feat: add --keep-original-ids to insights and dashboards#2

Merged
janmatzek merged 1 commit into
gooddata:masterfrom
janmatzek:jmat/allow-migration-of-objects-with-original-ids
May 20, 2026
Merged

feat: add --keep-original-ids to insights and dashboards#2
janmatzek merged 1 commit into
gooddata:masterfrom
janmatzek:jmat/allow-migration-of-objects-with-original-ids

Conversation

@janmatzek
Copy link
Copy Markdown
Contributor

No description provided.

@janmatzek janmatzek force-pushed the jmat/allow-migration-of-objects-with-original-ids branch from 42d7f4e to 8dc4c66 Compare May 13, 2026 12:37
@janmatzek janmatzek force-pushed the jmat/allow-migration-of-objects-with-original-ids branch from 8dc4c66 to 6155990 Compare May 13, 2026 12:43
@janmatzek janmatzek marked this pull request as ready for review May 14, 2026 12:48
@Tomkess
Copy link
Copy Markdown

Tomkess commented May 18, 2026

@janmatzek Quick question on IDs: with --keep-original-ids, report migration now seems to use the raw legacy identifier directly and skip the usual report-specific ID prefix. Was that intentional? Just want to make sure we’re OK with the possible collision/behavior change here.

@janmatzek
Copy link
Copy Markdown
Contributor Author

@janmatzek Quick question on IDs: with --keep-original-ids, report migration now seems to use the raw legacy identifier directly and skip the usual report-specific ID prefix. Was that intentional? Just want to make sure we’re OK with the possible collision/behavior change here.

@Tomkess Yes, intentional, happens for both reports and pixel perfect dashboards. It has side effects when the prefixes are omitted, they are mentioned in the readme file.

This covers a specific customer use case where retaining the IDs is requested to keep some mapping consistent on customer side.

@janmatzek janmatzek enabled auto-merge (rebase) May 20, 2026 10:19
@janmatzek janmatzek merged commit 4238427 into gooddata:master May 20, 2026
1 check passed
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.

2 participants