Skip to content

NIFI-13980: Edit Parameter Context action needs to be completed even …#9501

Merged
scottyaslan merged 1 commit into
apache:mainfrom
mcgilman:NIFI-13980
Nov 9, 2024
Merged

NIFI-13980: Edit Parameter Context action needs to be completed even …#9501
scottyaslan merged 1 commit into
apache:mainfrom
mcgilman:NIFI-13980

Conversation

@mcgilman
Copy link
Copy Markdown
Contributor

@mcgilman mcgilman commented Nov 7, 2024

…when the user is routed away by following a link to a referencing component.

To reproduce the issue...

  • Create a Parameter Context with a Parameter
  • Associate the Parameter Context with the current Process Group
  • Reference the Parameter in a Processor and save it.
  • Open the Processor configuration and in the Property Table select go to Parameter where it's referenced.
  • Edit the Parameter value and Apply.
  • Let the update complete.
  • Rather than closing the dialog, select the link to the referencing component shown in the dialog.
  • Open the Processor configuration and go to the Parameter again.

Without this fix, the previous update response is still shown in the dialog. With this fix, the user will be presented with the ability to reconfigure the Parameter.

…when the user is routed away by following a link to a referencing component.
@scottyaslan
Copy link
Copy Markdown
Contributor

Reviewing...

Copy link
Copy Markdown
Contributor

@scottyaslan scottyaslan left a comment

Choose a reason for hiding this comment

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

LGTM

@scottyaslan scottyaslan merged commit 21791b0 into apache:main Nov 9, 2024
KalmanJantner pushed a commit to KalmanJantner/nifi that referenced this pull request Feb 11, 2025
…when the user is routed away by following a link to a referencing component. (apache#9501)

(cherry picked from commit 21791b0)
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