-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NIFI-12761] refresh RPG #8401
[NIFI-12761] refresh RPG #8401
Conversation
Will review... |
There was a problem hiding this 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 @scottyaslan! Looking good, but I did note a few issues below.
...eb-frontend/src/main/nifi/src/app/pages/flow-designer/service/canvas-context-menu.service.ts
Show resolved
Hide resolved
...i-web/nifi-web-frontend/src/main/nifi/src/app/pages/flow-designer/state/flow/flow.effects.ts
Outdated
Show resolved
Hide resolved
...i-web/nifi-web-frontend/src/main/nifi/src/app/pages/flow-designer/state/flow/flow.effects.ts
Outdated
Show resolved
Hide resolved
852a92f
to
2de3c91
Compare
There was a problem hiding this 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 @scottyaslan! Looks great. Just a couple minor things below.
...i-web/nifi-web-frontend/src/main/nifi/src/app/pages/flow-designer/state/flow/flow.effects.ts
Outdated
Show resolved
Hide resolved
...i-web/nifi-web-frontend/src/main/nifi/src/app/pages/flow-designer/state/flow/flow.effects.ts
Outdated
Show resolved
Hide resolved
91f35a4
to
53c1da1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @scottyaslan! +1 will merge once CI completes.
NIFI-12761