GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1428
[NIFI-3359] Modularize all of nifi-web-ui except canvas directory
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1429
will review...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1429
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1438
Reviewing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1438#discussion_r98239800
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/provenance/nf-provenance-table.js
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1438
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1483
Will review...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1486
Will review
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1487
[NIFI-3360] Modularize Canvas
Most of this PR simply adds UMD (Universal Module Definition) boilerplate
around NIFI's existing modules and re-orders the loading of the scripts on the
pa
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1483
Thanks @mcgilman this has been merged to master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1488
[NIFI-3410] include processor-configuration.css in summary page
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1489
[NIFI-3305] update hover text for change color on the operate pallete
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1486
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1495
[NIFI-3450] update upload template dialog browse button
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1488
@mcgilman I opted to move the styles in question to processor-details.css
and updated this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1498
[NIFI-3458] increase processor preview height to avoid bottom border â¦
â¦being cut off
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1500
[NIFI-3459] remove blue highlighting from bread crumbs
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1348
@YolandaMDavis I have reviewed the UI portion of this PR and LGTM! +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1506
[NIFI-3360] resolve clashing variable name for connection and nf.Connâ¦
â¦ection
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of
Github user scottyaslan closed the pull request at:
https://github.com/apache/nifi/pull/1082
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1082
@mcgilman @joewitt it appears as though this approach does something to
adjust the zoom behavior but is not solving the root of the problem. I am
closing this PR and cancelling the patch. We will
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1512
reviewing...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1512
Thanks @andrewmlim this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1516
[NIFI-3181] add operate palette actions to context menu
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1519
[NIFI3501] Upgrade D3 to 3.5.17 and allow npm to pull it down at builâ¦
â¦d time
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1519
@mcgilman sure thing!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1519
@mcgilman this PR has been updated to include the copyright date.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1533
[NIFI-3358] update naming conventions to avoid conflicts
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1534
[NIFI-3038] Update cluster summary dialog layout
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1535
[NIFI-2891] persist selected tab for connection configuration dialog,â¦
⦠except for when creating a new connection the dialog will open with the
details tab visible since users will need to
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1538
[NIFI-2941] allow users to click checkbox labels to toggle the checkbox
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1540
[NIFI-3358] remove duplicate calls to nfProcessGroupConfiguration.shoâ¦
â¦wConfiguration()
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1538
Do you mean moving the on click call back function:
function () {$(this).parent().find('.nf-checkbox').click();
to somewhere like nfCanvasUtils?
---
If your project
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1538
OK...I have update this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1538
:)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1545
[NIFI-3465] change CS and RT warning tooltip viewports to a visible aâ¦
â¦nd connected DOM element.
Thank you for submitting a contribution to Apache NiFi.
In order to
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1546
[NIFI-3439] when users type into sensitive property fields in NiFI, tâ¦
â¦he values on the screen/browser should be masked *** unless specified by
user to show plain text.
Thank you
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1559
[NIFI-3035] add deep linking URL to canvas components and PGs
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1560
[NIFI-3550] inject nf.ConnectionConfiguration into the nf.Cannection â¦
â¦module during nf.Graph init
Thank you for submitting a contribution to Apache NiFi.
In order to
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1591
Reviewing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1590
Reviewing...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1591
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1590
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1593
Reviewing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1593
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105746348
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/pom.xml
---
@@ -440,6 +440,7
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105746551
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas-bootstrap.js
---
@@ -339,6
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105746988
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-component-version.js
---
@@ -0,0
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105747481
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-actions.js
---
@@ -1214,6 +1217,23
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105747710
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-component-version.js
---
@@ -0,0
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105973908
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/resources/filters/canvas.properties
---
@@ -28,6 +28,7
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1585#discussion_r105977072
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/jquery/propertytable/jquery.propertytable.js
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1594
Reviewing...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1594
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1599
Reviewing...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/1599
Thanks @mcgilman this has been merged to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/1600
[NIFI-2697] revert transmission switch toggle on failed update request
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2607
@mcgilman I have reviewed the client side code and I am a +1 to merge once
you have made the updates to the messages as @moranr suggested. Thanks for this
contribution!
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2612
Reviewing...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2612
Thanks @andrewmlim this has been merged to master.
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-registry/pull/109
[NIFIREG-161] selectable bucket and flow UUIDs in flow details panel
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/scottyaslan
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2613
Reviewing...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2613
Thanks @vion1 this has been merged to master.
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-registry/pull/111
[NIFIREG-161] add vendor prefix to user-select style
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/scottyaslan/nifi-registry
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2620
Will Review...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2620
Thanks @andrewmlim this has been merged to master.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2607
Looks good!
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2624
Reviewing...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2633
+1 for the UI code.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-registry/pull/113
Reviewing...
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-registry/pull/115
[NIFIREG-170] upgrade hoek.js
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/scottyaslan/nifi-registry NIFIREG-170
Alternatively
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-registry/pull/116
[NIFIREG-171] update moment.js
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/scottyaslan/nifi-registry NIFIREG-171
Alternatively
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2688
Will review...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2688
Thanks @mcgilman this has been merged to master.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2692
Reviewing...
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-fds/pull/2
[NIFI-5198] update tooling versions
Thank you for submitting a contribution to Apache NiFi Fluid Design System.
In order to streamline the review of the contribution we ask you
to
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2692
Thanks @mcgilman this has been merged to master.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2716
Reviewing...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2716
Thanks @andrewmlim this has been merged to master.
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi/pull/2721
[NIFI-5215] Upgrade AngularJS to v1.7.0. Also, adding package-lock.jsâ¦
â¦on to source control
Thank you for submitting a contribution to Apache NiFi.
In order to streamline
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2722#discussion_r189688839
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-controller-services.js
Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2722#discussion_r189689454
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-settings.js
---
@@ -264,12 +264,20
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2722
Thanks @mcgilman this has been merged to master.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2719
Reviewing...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2719
Thanks @andrewmlim this has been merged to master.
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-fds/pull/3
[NIFI-5219] update project name to NiFi Flow Design System
Thank you for submitting a contribution to Apache NiFi Fluid Design System.
In order to streamline the review of the
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-fds/pull/2
The karma files are from the Angular Quickstart MIT licensed project and is
already listed in the LICENSE file.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-fds/pull/2
@joewitt I have clarified which items from the Angular Quickstart and add
headers to each of the files including the package-lock.json file which is
intended to be committed into source
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-fds/pull/2
^ that was taken from the package-lock.json documentation on npm at:
https://docs.npmjs.com/files/package-lock.json
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-fds/pull/2
@joewitt I believe you need to run `npm cache verify` to get rid of those
'stale package data' warnings.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2740
Thanks @mcgilman this is a nice addition to the cluster UX. This has been
merged to master!
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-registry/pull/120
Will review...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2757
Reviewing...
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2757
Thanks @mcgilman this has been merged to master.
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-fds/pull/4
[NIFI-5216] facilitate npm publish
Thank you for submitting a contribution to Apache NiFi Flow Design System.
In order to streamline the review of the contribution we ask you
to
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2766
Reviewing...
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-registry/pull/123
[NIFIREG-39] leverage npm @nifi-fds/core
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/scottyaslan/nifi-registry NIFIREG-39
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi-registry/pull/123
Please do not merge.
---
Github user scottyaslan commented on the issue:
https://github.com/apache/nifi/pull/2766
@mcgilman I merged this PR to master...thanks for the awesome contrib!
---
GitHub user scottyaslan opened a pull request:
https://github.com/apache/nifi-fds/pull/5
add flow_design_styles.css to demo-app index.html
Thank you for submitting a contribution to Apache NiFi Flow Design System.
In order to streamline the review of the contribution we ask
1 - 100 of 389 matches
Mail list logo