Skip to content

UIEH-1575: Add Connected Tasks/Jobs to eHoldings record views. - #1853

Merged
Dmytro-Melnyshyn merged 4 commits into
masterfrom
UIEH-1575
Sep 25, 2026
Merged

Dmytro-Melnyshyn merged 4 commits into
masterfrom
UIEH-1575

Conversation

@Dmytro-Melnyshyn

@Dmytro-Melnyshyn Dmytro-Melnyshyn commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Purpose

Add the Connected Tasks/Jobs button (ConnectedTasksJobsButton) and pane (ConnectedTasksJobsPane) to Provider, Package, and Title detail pages through <Pluggable>. Pass record context, including the name displayed during Task creation.

Description

The ConnectedTasksJobsPane uses layer=connected-tasks-jobs in the URL. Opening and closing it adds browser-history entries for the same record. Previously, the record's "X" button could follow those entries and reopen the ConnectedTasksJobsPane instead of returning to the page that opened the record. Visits to edit pages and other apps add further navigation steps.

The back-navigation changes account for these entries:

  • Read the latest recorded history when "X" is clicked.
  • Resolve POP events to exclude visits already left behind by browser Back.
  • Treat a record's view and /edit URLs as the same record.
  • Use leavingEholdings and the record path to recognize visits to other apps followed by a return to that record.
  • Find the preceding different eHoldings page and count only PUSH events, since REPLACE does not add a history step.
  • Navigate back by that count, preserving the destination's original URL and search parameters.

The calculation is isolated in getRecordCloseNavigation, and useHistoryBack uses the result. When pane history has no reliable parent, navigation falls back to eHoldings search.

Refs

ERM-4158

@Dmytro-Melnyshyn
Dmytro-Melnyshyn requested a review from a team as a code owner September 18, 2026 13:12
@Dmytro-Melnyshyn
Dmytro-Melnyshyn requested review from a team and NikitaSedyx September 18, 2026 13:12
@github-actions

github-actions Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Jest Unit Test Results

    1 files  ± 0    205 suites  +1   5m 5s ⏱️ + 2m 19s
1 199 tests +29  1 196 ✅ +29  3 💤 ±0  0 ❌ ±0 
1 206 runs  +29  1 203 ✅ +29  3 💤 ±0  0 ❌ ±0 

Results for commit a84a693. ± Comparison against base commit 5c519c8.

♻️ This comment has been updated with latest results.

@Dmytro-Melnyshyn
Dmytro-Melnyshyn requested review from a team and removed request for a team September 21, 2026 12:30
@sonarqubecloud

Copy link
Copy Markdown

@Dmytro-Melnyshyn
Dmytro-Melnyshyn merged commit 9d397d6 into master Sep 25, 2026
17 checks passed
@Dmytro-Melnyshyn
Dmytro-Melnyshyn deleted the UIEH-1575 branch September 25, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants