-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix: Update Remote URLs in NiFi for fetching data #115
Conversation
IMHO it's better to merge into main to get the fix right now. There is no need to wait for SDP 24.11, as we are not using any new product versions or features of it |
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.
LGTM
I'll update the |
I though we are going to merge next into main eventually? Wouldn't this cause merge conflicts? |
Ahh or merge main into next? That would make sense I guess |
I'll cherry-pick this into the release-24.7 branch. |
* fix: Update Remote URLs in NiFi for fetching data * docs: update nifi screenshot to one without errors
Fix for #62
Also includes an updated screenshot for the documentation, which was showing errors in NiFi because of this.