Skip to content
Snippets Groups Projects

Custom cache key

Merged Coby Sher requested to merge issue/api_client-3389581:custom-cache-key into canary
1 unresolved thread

Adds a cacheKey option to the BaseRequestOptions which is passed to createCacheKey. Tests updated too.

Merge request reports

Pipeline #192833 passed

Pipeline passed for 371f8e28 on issue:custom-cache-key

Approved by
License Compliance test metrics results are being parsed

Merged by Brian PerryBrian Perry 8 months ago (Jun 6, 2024 3:04pm UTC)

Merge details

  • Changes merged into canary with 4aeef422 (commits were squashed).
  • Did not delete the source branch.
  • Auto-merge enabled

Pipeline #192843 passed

Pipeline passed for 4aeef422 on canary

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
109 110 static createCacheKey({
110 111 localeSegment,
111 112 path,
113 cacheKey,
112 114 }: {
113 115 localeSegment?: string;
114 path: string;
116 path?: string;
117 cacheKey?: string;
115 118 }) {
119 if (cacheKey) {
120 return cacheKey;
121 }
122
123 if (!path) {
  • Brian Perry approved this merge request

    approved this merge request

  • Brian Perry enabled an automatic merge when the pipeline for 371f8e28 succeeds

    enabled an automatic merge when the pipeline for 371f8e28 succeeds

  • merged

  • Please register or sign in to reply
    Loading