tradingstrategy-ai / frontend

Web frontend for TradingStrategy.ai
https://tradingstrategy.ai
131 stars 23 forks source link

Deprecated strategies with replacement strategy pointer #798

Closed kenkunz closed 3 months ago

kenkunz commented 3 months ago

Goal

As a Trading Strategy admin, I want to be able to mark an existing "live" strategy as "deprecated", so that end users of our platform will be notified when viewing the strategy and redirected to an updated version.

Acceptance criteria

  1. A Trading Strategy admin can indicate that a strategy is deprecated, as well as provide a replacement strategy pointer, in the frontend strategy configuration.
  2. A deprecated strategy displays a conspicuous deprecation warning at the top of the strategy overview page, which includes a link to the replacement strategy. (e.g,: "Strategy deprecated, new version deployed here.")
  3. The ( Deposit ) button is disabled on deprecated strategies.
  4. A deprecated strategy includes a "deprecated" badge on the strategy index page, with an appropriate tooltip (see mock up below)
  5. A deprecated strategy is never displayed on the front page
  6. A deprecated strategy is positioned after other "live" strategies in the sort order
    • this turns out to be overly complicated, since we already sort "disconnected" strategies to the end, and already rely on two other methods to indicate sort order: the order within frontend config and the optional sort_priority attribute from strategy metadata.
    • work-around is to just re-order the strategy within frontend config to the right position

Visuals

Deprecated strategy badge

deprecated badge