pixiebrix-extension icon indicating copy to clipboard operation
pixiebrix-extension copied to clipboard

Refactor fetch deployment mod definitions axios call to use RTK Query

Open johnnymetz opened this issue 1 year ago • 1 comments

User Story

As a frontend developer, I an API call to fetch deployment mod definitions to use RTK, so we get the RTK query benefits (caching) and consistency with the rest of the codebase

Motivation

RTK query is better than using axios directly

Acceptance Criteria

  • Update this axios call to use RTK query:
  • Note, we need to keep the fetchDeploymentModDefinitions function because it's used in the background script, see deploymentUpdater.ts (unless we somehow figure out how to use RTK query in the bg script)

johnnymetz avatar Mar 06 '24 20:03 johnnymetz

, so we get the RTK query benefits (caching

As a reminder, we still get browser-level caching in the current world

twschiller avatar Mar 07 '24 17:03 twschiller

This issue will be closed in 7 days unless the stale label is removed, or a comment is added to the issue.

github-actions[bot] avatar Jun 23 '24 00:06 github-actions[bot]

This issue was closed because it has been stale for 7 days with no activity.

github-actions[bot] avatar Jun 30 '24 00:06 github-actions[bot]