[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

JPercivall
GitHub user danbress opened a pull request:

    https://github.com/apache/incubator-nifi/pull/29

    NIFI-344 - Remote processor group lets you specify protocols other than http/https

    On the front end, providing a placeHolder hint as to what the expected value looks like
    On the back end validating that the user specified url(on create or update) is http://something or https://something

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/danbress/incubator-nifi NIFI-344

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-nifi/pull/29.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #29
   
----
commit 09090825dea376abd5a4c962fa2034e952b7dca1
Author: danbress <[hidden email]>
Date:   2015-02-17T17:39:20Z

    NIFI-344 - Validating remote process group URI in the backend

commit 10b5b3c90702115698b9d88180a806a44e96dda1
Author: danbress <[hidden email]>
Date:   2015-02-19T02:55:47Z

    NIFI-344 - Defaulting new Remote Process Group value

commit 750c4ebb6c1d2f91e408ae7d8cbaf252cbcef30a
Author: danbress <[hidden email]>
Date:   2015-02-19T23:15:29Z

    NIFI-344 improving error message, and setting a placeHolder for the
    field

----


---
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 enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

JPercivall
Github user mcgilman commented on the pull request:

    https://github.com/apache/incubator-nifi/pull/29#issuecomment-75228699
 
    Cool! Didn't know about the placeholder attribute. I've been doing that myself using focus/blur events and css. The only downside is that it isn't supported in IE9 (which technically we do support). However, I think it's a fair tradeoff, so I'll put in a ticket to switch usage to placeholder everywhere.


---
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 enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

JPercivall
In reply to this post by JPercivall
Github user danbress commented on the pull request:

    https://github.com/apache/incubator-nifi/pull/29#issuecomment-75235591
 
    Thanks.  I wanted to ask you about the placeholder attribute, since it is new to HTML5, and wasn't sure how far back we were supporting.
   
    Do we have a list of the minimum browsers/standards that are required to run the app?
   



---
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 enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

JPercivall
In reply to this post by JPercivall
Github user danbress commented on the pull request:

    https://github.com/apache/incubator-nifi/pull/29#issuecomment-75236952
 
    Just poked through the website, didn't see what I was looking for in terms of "minimum system requirements" to run the software.  Just something about maven and JDK on the development page.  So I created a ticket NIFI-370


---
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 enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

JPercivall
In reply to this post by JPercivall
Github user mcgilman commented on the pull request:

    https://github.com/apache/incubator-nifi/pull/29#issuecomment-75237020
 
    Those details used to be in the user guide but I don't see it in the version that was put into 0.0.1. It should go where ever we've documented other minimum requirements (JVM version, etc).
   
    To answer the question directly, the limiting factor for browsers is SVG support. IE introduced support for SVG in IE 9.  


---
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 enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] incubator-nifi pull request: NIFI-344 - Remote processor group let...

JPercivall
In reply to this post by JPercivall
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-nifi/pull/29


---
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 enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---