theatre icon indicating copy to clipboard operation
theatre copied to clipboard

Make reason mandatory when interactive

Open doomholderz opened this issue 1 year ago • 1 comments

Making reason entry mandatory on interactive console creations

doomholderz avatar Nov 07 '24 14:11 doomholderz

While I'm no longer a maintainer on this repo, I can provide some historical context:

The runner package in this repo was designed to be relatively un-opinionated, mostly as an inspiration and something that you could use when developing against theatre. At GoCardless (and incident.io) we imported this package into our own CLI, and only in those CLI's would we encode organisation-specific logic like requiring a reason.

I don't think it's practical for anyone to be using the runner CLI here for their own organisation, given that it's not packaged and distributed nicely, hence I'd be against putting org-specific requirements into it.

benwh avatar Dec 08 '24 17:12 benwh

Closing as:

  • I agree with comment from @benwh - this requirement should be implemented in a consumer of the package;
  • I think this is already accomplished on our side via the incident ID requirement in Utopia cli;
  • Project structure has changed substantially since Theatre v5 release;

0x0013 avatar Dec 05 '25 10:12 0x0013