box / boxcli

A command line interface for interacting with the Box API.
https://developer.box.com
Apache License 2.0
228 stars 59 forks source link

event:poll for user-event ignores polling-interval #418

Closed kobay closed 1 year ago

kobay commented 2 years ago

Description of the Issue

CLI creates an option field pollingInterval when polling-interval is given. This doesn't work with user event, because the parameter name in getEventStream is fetchInterval.

https://github.com/box/boxcli/blob/f1bd958cd1142bb4328e8cfad72995b55236ec04/src/commands/events/poll.js#L23 https://github.com/box/box-node-sdk/blob/bf15111717cce9c97f5597c7140c639183dd3b95/src/managers/events.ts#L265

Steps to Reproduce

--polling-interval has no effect in the following commands.

box events:poll --polling-interval=300 box events:poll --polling-interval=10

mwwoda commented 2 years ago

Hi @kobay Thanks for reporting this bug. I see that indeed the parameter is passed incorrectly. I have created a ticket in our internal system (SDK-2725). We'll keep you posted if there are any updates on this issue.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not been updated in the last 30 days. It will be closed if no further activity occurs within the next 7 days. Feel free to reach out or mention Box SDK team member for further help and resources if they are needed.