Delinquency notifications

[sbosio]

Description

Here we're adding a new feature to the API client implementation in @heroku-cli/command where an additional request is done to Particleboard whenever the request URL matches the Account Info endpoint from Platform API (GET /account), or any endpoint in the Team namespace (GET /teams/:team_id/*).

For this feature we're introducing a new API client for Particleboard with its required configuration and changes to the API client request function implementation to match URLs against the patterns of interest and send requests to Particleboard if required to get the delinquency info.

Reference RFC

SOC2 Compliance

GUS Work item

[justinwilaby]

I believe the deps.ts file was created by someone unfamiliar with how node requires dependencies and was intended to cache these for faster access. However, node has been doing this internally for a decade and a half which makes this strategy superfluous. Is there an opportunity to remove this file in favor of the standard import ?

[zwhitfield3]

This is a well organized and thoroughly tested PR. I left a few comments that are non-blocking. Great work @sbosio!

[sbosio]

Is there an opportunity to remove this file in favor of the standard import ?

I must admit that it looked weird to me and I didn't feel strong to make this change as part of the PR, because it felt like too much unrelated.

I will add a card for this to our backlog, though to ensure we get to it eventually.

Thanks for flagging it @justinwilaby. I don't have a strong background on JS / TS yet to recognize these anti-patterns.

[sbosio]

@justinwilaby Added this work item to GUS for tracking removal of the deps.ts file.