Skip to content

NIFI-13017: Replace JoltTransformJSON custom UI#9117

Merged
rfellows merged 5 commits intoapache:mainfrom
mcgilman:NIFI-13017
Aug 6, 2024
Merged

NIFI-13017: Replace JoltTransformJSON custom UI#9117
rfellows merged 5 commits intoapache:mainfrom
mcgilman:NIFI-13017

Conversation

@mcgilman
Copy link
Contributor

NIFI-13017:

  • Remove old JoltTransformJSON custom UI.
  • Update build to use new JoltTransformJSON custom UI.
  • Fix revision query parameter.

Copy link
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.

A bit unrelated but I noticed that there is an error message the flashes on initial load of the Jolt Custom UI. This is a separate issue and I logged https://issues.apache.org/jira/browse/NIFI-13589 to address that.

@mcgilman mcgilman added the ui Pull requests for work relating to the user interface label Jul 29, 2024
mcgilman added 5 commits July 31, 2024 15:26
- Remove old JoltTransformJSON custom UI.
- Update build to use new JoltTransformJSON custom UI.
- Fix revision query parameter.
- Excluding jolt transform json ui in workflow ci.
- Excluding jolt nar in workflow ci.
- Excluding jolt ui war from code coverage.
- Incorporating nifi-web-servlet-shared.
- Moving QueryStringToFragmentFilter to nifi-web-servlet-shared.
Copy link
Contributor

@rfellows rfellows left a comment

Choose a reason for hiding this comment

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

Excellent work @mcgilman! Nice to finally get rid of AngularJS!

@scottyaslan
Copy link
Contributor

I verified that purple mode works well and that the browser back button navigation is fixed. Nice work @mcgilman!

@rfellows rfellows merged commit da6c9c4 into apache:main Aug 6, 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.

3 participants