Skip to content

FOUR-32196: Add importing user context to DevLink bundle imports#8903

Open
eiresendez wants to merge 2 commits into
developfrom
epic/FOUR-32196
Open

FOUR-32196: Add importing user context to DevLink bundle imports#8903
eiresendez wants to merge 2 commits into
developfrom
epic/FOUR-32196

Conversation

@eiresendez

@eiresendez eiresendez commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

DevLink bundle imports do not expose the destination user who initiated the installation to asset importers. This prevents package importers from assigning destination-owned resources correctly because identifiers such as a Dashboard user_id belong to the source instance.

To reproduce:

  1. In instance A, create a Display Screen and a Custom Dashboard owned by a user from A.
  2. Add the Dashboard to a DevLink bundle and publish it.
  3. In instance B, install or reinstall the bundle as an administrator whose user ID differs from the source owner.
  4. The importer only receives the import mode and cannot reliably assign the administrator from B as the resource owner.

Solution

  • Add an optional, backward-compatible importingUserId context to Import/Export Options.
  • Populate importingUserId from the DevLink Logger when installing bundle setting payloads.
  • Propagate the same context through regular bundle installation and reinstall operations.
  • Add coverage for the optional Options contract and reinstall context propagation.

How to Test

Run the focused core tests:

vendor/bin/phpunit tests/Feature/ImportExport/OptionsTest.php tests/Model/BundleTest.php tests/Model/DevLinkTest.php --no-coverage

Result: 10 tests and 33 assertions passed.

The package PR linked below contains the end-to-end Custom Dashboard installation, reinstall, and copy coverage.

Related Tickets & Packages

ci:deploy
ci:package-dynamic-ui:epic/FOUR-32196

@cursor

cursor Bot commented Jul 10, 2026

Copy link
Copy Markdown

Bugbot is not enabled for this team, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@processmaker-sonarqube

Copy link
Copy Markdown

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://tenant-1.ci-992dbc27d4.engk8s.processmaker.net

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-992dbc27d4.engk8s.processmaker.net

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