Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

As a User, I can launch a request on any reader using a pre-defined date_range option #82

Closed
gabrielleberanger opened this issue Jan 14, 2021 · 1 comment · Fixed by #83
Closed

As a User, I can launch a request on any reader using a pre-defined date_range option #82

gabrielleberanger opened this issue Jan 14, 2021 · 1 comment · Fixed by #83
Assignees
@benoitgoujon
Labels
new feature Creating a new feature P1 1st priority

Comments

@gabrielleberanger
Copy link
Contributor

WHY
Currently, only specific readers accept a --date_range parameter (it depends of the source API, that offers it or not).

Such a pre-defined --date_range option can be very convenient, in particular when conducting tests. In this case, you just have to specify --date-range PREVIOUS_WEEK for instance, instead of providing a specific --start-date and --end-date for your request.

HOW
Make the --date_range option available to all readers, including when it is not initially provided by the source API.
@gabrielleberanger gabrielleberanger added new feature Creating a new feature P1 1st priority labels Jan 14, 2021
@benoitgoujon
Copy link
Contributor

Also see issue #67 for a description of how each reader handles date parameters.

@benoitgoujon benoitgoujon mentioned this issue Jan 15, 2021
Merged
1 task
@benoitgoujon benoitgoujon linked a pull request Jan 15, 2021 that will close this issue
Feat/date range option for all readers #83
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benoitgoujon benoitgoujon

Labels
new feature Creating a new feature P1 1st priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feat/date range option for all readers artefactory/artefactory-connectors-kit
2 participants
@gabrielleberanger @benoitgoujon