Skip to content

[NIFI-12622] - Parameter Providers - listing table#8298

Merged
mcgilman merged 4 commits intoapache:mainfrom
rfellows:NIFI-12622-parameterProviders
Jan 26, 2024
Merged

[NIFI-12622] - Parameter Providers - listing table#8298
mcgilman merged 4 commits intoapache:mainfrom
rfellows:NIFI-12622-parameterProviders

Conversation

@rfellows
Copy link
Copy Markdown
Contributor

@rfellows rfellows commented Jan 24, 2024

NIFI-12622

  • Add a new Parameter Provider
  • Delete parameter provider
  • refactor to reduce duplicate code when creating new properties
  • support editing parameter providers
  • refactored inline service creation into PropertyTableHelperService
  • added parameter provider linking to access policies.

There was a minor backend change to include the time the parameter provider listing was returned.

This does not add support for "fetch". That will be a large enough effort to warrant its own PR - see NIFI-12665

@rfellows rfellows added the ui Pull requests for work relating to the user interface label Jan 24, 2024
* Add a new Parameter Provider
* Delete parameter provider
* refactor to reduce duplicate code when creating new properties
* support editing parameter providers
* refactored inline service creation into PropertyTableHelperService
* added parameter provider linking to access policies.
@rfellows rfellows force-pushed the NIFI-12622-parameterProviders branch from f64fa5a to f2bcfa2 Compare January 25, 2024 20:35
Copy link
Copy Markdown
Contributor

@mcgilman mcgilman left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @rfellows! Looking good... just a couple minor things below.

…ented in loads of places. replaced all occurrences to the nifiCommon implementation.
Copy link
Copy Markdown
Contributor

@mcgilman mcgilman left a comment

Choose a reason for hiding this comment

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

Thanks for the updates @rfellows! +1

@mcgilman mcgilman merged commit 0e87032 into apache:main Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui Pull requests for work relating to the user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants