Skip to content

Adding to the Stripe CLI - reporting.report_run.succeeded#823

Closed
hotero001 wants to merge 7 commits into
stripe:masterfrom
hotero001:master
Closed

Adding to the Stripe CLI - reporting.report_run.succeeded#823
hotero001 wants to merge 7 commits into
stripe:masterfrom
hotero001:master

Conversation

@hotero001
Copy link
Copy Markdown

@hotero001 hotero001 commented Feb 18, 2022

Reviewers

r? @
cc @stripe/developer-products

Summary

Could you review this request, please? A User requested to have reporting.report_run.succeeded & reporting.report_run.failed added to the Stripe CLI, so they may perform $stripe trigger reporting.report_run.succeeded & $stripe trigger reporting.report_run.failed

reporting.report_run.succeeded add event to Stripe CLI
reporting.report_run.succeeded added to Stripe CLI
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Feb 18, 2022

CLA assistant check
All committers have signed the CLA.

@tomer-stripe
Copy link
Copy Markdown
Collaborator

hi @hotero001! Sorry for the delay, I just pulled the PRs and was not able to get this working correctly.

The success one fails for me:

➜ stripe-dev trigger reporting.report_run.succeeded
Setting up fixture for: reporting
Running fixture for: reporting
Trigger failed: Request failed, status=400, body={
  "error": {
    "code": "parameter_missing",
    "doc_url": "https://stripe.com/docs/error-codes/parameter-missing",
    "message": "Missing required param: parameters[interval_end].",
    "param": "parameters[interval_end]",
    "type": "invalid_request_error"
  }
}


exit status 1

The failed trigger fires off correctly but I think because there's a param failure it doesn't actually trigger the failed event.

@hotero001 hotero001 closed this Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants