feat(ajax): add interceptors for parsed response JSON objects

[bashmish]

What I did

1. added addResponseJsonInterceptor to be able to intercept parsed response JSON objects
2. added docs and tests for it

[changeset-bot]

🦋 Changeset detected

Latest commit: 3ed0929

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@lion/ajax	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[CLAassistant]

All committers have signed the CLA.

[tlouisse]

@ing-athirlwall could you maybe have a look?

[ing-athirlwall]

LGTM. Perhaps use "forEach" instead of "for..of" when iterating through the interceptors but that's just a niggle.

[bashmish]

LGTM. Perhaps use "forEach" instead of "for..of" when iterating through the interceptors but that's just a niggle.

as discussed in the chat, it was implemented the same way as other interceptor handlers, so should be good for code consistency :) e.g. this one:

  /**
   * @param {Response} response
   * @returns {Promise<Response>}
   */
  async __interceptResponse(response) {
    let interceptedResponse = response;
    for (const intercept of this._responseInterceptors) {
      // In this instance we actually do want to await for each sequence
      // eslint-disable-next-line no-await-in-loop
      interceptedResponse = await intercept(/** @type {CacheResponse} */ (interceptedResponse));
    }
    return interceptedResponse;
  }

[ing-athirlwall]

Fair enough .. like the feature, let's release it.

[tlouisse]

Approved and merged. Sorry for the delay (I thought you were able to approve this as well @ing-athirlwall)