deployments icon indicating copy to clipboard operation
deployments copied to clipboard

fix(delete-env): proper warning on 404

Open mtfurlan opened this issue 2 years ago • 1 comments

When we get a 404 deleting an environment there should be a better warning than "unexpected error encountered: HttpError: Not Found" Document in the readme that the user for the PAT must have admin on the repo, not just write or manage.

mtfurlan avatar Nov 02 '23 21:11 mtfurlan

Thanks for the PR. It helped me to resolve why my env deletions are failing 👍

mmkoles avatar Nov 10 '23 15:11 mmkoles

@mtfurlan I've been strugging with getting delete-env to work. Does this work with fine-grained personal access tokens or just the classic tokens?

sguha00 avatar Mar 04 '24 07:03 sguha00

Yeah, this repo works with PATs, but as I have in the readme change, the PAT needs repo scope permissions, and the owner of the PAT needs to be admin on the repo.

mtfurlan avatar Mar 05 '24 17:03 mtfurlan