clerk / clerk-sdk-node

Access the Clerk Backend API from Node.js
MIT License
7 stars 4 forks source link

Add order_by parameter to getUserList #42

Closed lluisgassovillarejo closed 2 years ago

lluisgassovillarejo commented 2 years ago

Hello,

I am working on an integration to export users incrementally from Clerks. To do that, it would be great if we could sort users by the last update date. The order_by parameter was recently added to the API, but the node SDK hasn't added it yet. For what I can see, it is a matter of adding it to the UserListParams interface.

Would it be possible to add it? If that helps, I can create a PR with that update.

Thanks!

SokratisVidros commented 2 years ago

Hey @lluisgassovillarejo. Thanks for bringing this to our attention. We will push a fix later today.

lluisgassovillarejo commented 2 years ago

Awesome! Many thanks!

On 16 Nov 2021, at 11:34, Sokratis Vidros @.***> wrote:

Hey @lluisgassovillarejo https://github.com/lluisgassovillarejo. Thanks for bringing this to our attention. We will push a fix later today.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/clerkinc/clerk-sdk-node/issues/42#issuecomment-970137022, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACP2R7TTIYUKYXPQTJZJDNTUMIXR5ANCNFSM5ID24HPA. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.