[GitHub] syncope pull request #63: When importing users, cannot import more than 200 ...

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

[GitHub] syncope pull request #63: When importing users, cannot import more than 200 ...

ilgrosso-2
GitHub user tlnd-fcurvat opened a pull request:

    https://github.com/apache/syncope/pull/63

    When importing users, cannot import more than 200 users.

    Hi,
    Found a bug when doing migration from syncope 1.2 to syncope 2.0.
    Cannot migrate more than 200 users.
    This "ugly" fix works for me, so i wanted to share.
    Regards,

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

    $ git pull https://github.com/tlnd-fcurvat/syncope fcurvat/2_0_X

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

    https://github.com/apache/syncope/pull/63.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 #63
   
----
commit 3e07c214c2d02276a9d66118bd495e2afc9c8522
Author: Frédéric Curvat <[hidden email]>
Date:   2017-09-20T12:50:43Z

    When importing users, cannot import more than 200 users.

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] syncope issue #63: When importing users, cannot import more than 200 users.

ilgrosso-2
Github user ilgrosso commented on the issue:

    https://github.com/apache/syncope/pull/63
 
    Hi @tlnd-fcurvat thanks for this PR.
   
    The fix looks good and I could merge it straight away, but I'd need first a couple of things:
   
    1. you should submit an [ICLA](http://syncope.apache.org/contributing.html#How_do_I_become_a_contributor_or_a_committer)
    1. you should open an issue on [JIRA](https://issues.apache.org/jira/projects/SYNCOPE) and possible reference its name (`SYNCOPE-XXXX`) in the commit message
   
    Thanks!


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] syncope issue #63: When importing users, cannot import more than 200 users.

ilgrosso-2
In reply to this post by ilgrosso-2
Github user ilgrosso commented on the issue:

    https://github.com/apache/syncope/pull/63
 
    I have just seen SYNCOPE-1211 thanks!


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] syncope issue #63: When importing users, cannot import more than 200 users.

ilgrosso-2
In reply to this post by ilgrosso-2
Github user tlnd-fcurvat commented on the issue:

    https://github.com/apache/syncope/pull/63
 
    Quite busy, i will submit ICLA tomorrow.


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] syncope pull request #63: When importing users, cannot import more than 200 ...

ilgrosso-2
In reply to this post by ilgrosso-2
Github user asfgit closed the pull request at:

    https://github.com/apache/syncope/pull/63


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] syncope issue #63: When importing users, cannot import more than 200 users.

ilgrosso-2
In reply to this post by ilgrosso-2
Github user ilgrosso commented on the issue:

    https://github.com/apache/syncope/pull/63
 
    Thanks @tlnd-fcurvat !


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] syncope issue #63: When importing users, cannot import more than 200 users.

ilgrosso-2
In reply to this post by ilgrosso-2
Github user tlnd-fcurvat commented on the issue:

    https://github.com/apache/syncope/pull/63
 
    my pleasure :+1:


---