Move endpoint cache to a new service.

Created on 2 June 2024, 7 months ago
Updated 7 June 2024, 7 months ago

Problem/Motivation

The responsibility of the iconify service should not include writing to the file system, although it should use a caching system.

Proposed resolution

* Create a new service for the cache
* Inject the cache service to the iconify service
* Create proper testing for the new service and clean the ones in the iconify service.

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇪🇸Spain alvar0hurtad0 Cáceres

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @alvar0hurtad0
  • 🇪🇸Spain alvar0hurtad0 Cáceres

    Problem/ motivation

    The responsibility of the iconify service should not include writing to the file system, although it should use a caching system.

    Proposed solution

    * Create a new service for the cache
    * Inject the cache service to the iconify service
    * Create proper testing for the new service and clean the ones in the iconify service.

  • First commit to issue fork.
  • Pipeline finished with Failed
    7 months ago
    Total: 738s
    #190667
  • Pipeline finished with Failed
    7 months ago
    Total: 151s
    #190678
  • Pipeline finished with Failed
    7 months ago
    Total: 149s
    #190696
  • Pipeline finished with Canceled
    7 months ago
    Total: 512s
    #190701
  • Pipeline finished with Failed
    7 months ago
    Total: 493s
    #190706
  • Pipeline finished with Failed
    7 months ago
    Total: 1034s
    #190712
  • Pipeline finished with Success
    7 months ago
    Total: 169s
    #190993
  • Pipeline finished with Success
    7 months ago
    Total: 145s
    #191000
  • Pipeline finished with Success
    7 months ago
    Total: 146s
    #191009
  • Pipeline finished with Success
    7 months ago
    Total: 144s
    #191585
  • Pipeline finished with Success
    7 months ago
    Total: 148s
    #191595
  • Pipeline finished with Success
    7 months ago
    Total: 151s
    #191602
  • Pipeline finished with Success
    7 months ago
    Total: 149s
    #191608
  • Status changed to Needs review 7 months ago
  • daniel_mm02 Burriana, Castellón

    Please check merge request

  • Status changed to Needs work 7 months ago
  • First commit to issue fork.
  • Pipeline finished with Success
    7 months ago
    Total: 166s
    #192499
  • Pipeline finished with Success
    7 months ago
    Total: 171s
    #192528
  • Pipeline finished with Skipped
    7 months ago
    #192543
  • Status changed to Fixed 7 months ago
  • 🇪🇸Spain alvar0hurtad0 Cáceres

    Thanks for your effort

  • 🇪🇸Spain gxleano Cáceres

    It will be part of release 1.0.1

  • Issue was unassigned.
  • 🇪🇸Spain gxleano Cáceres
  • Status changed to Fixed 7 months ago
  • 🇪🇸Spain gxleano Cáceres
Production build 0.71.5 2024