Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recalculate summary metrics after deposit/withdrawal #659

Open
kenkunz opened this issue Nov 7, 2023 · 0 comments
Open

Recalculate summary metrics after deposit/withdrawal #659

kenkunz opened this issue Nov 7, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request priority: P3 The lowest priority. A desired nice-to-have improvement, but can be postponed for several releases. size: S A small task that can be completed in less than a day

Comments

@kenkunz
Copy link

kenkunz commented Nov 7, 2023

Goal

As an investor, I want the strategy summary metrics (displayed on the strategy index page) to be updated promptly after making a deposit or withdrawal.

Acceptance Criteria

  • summary stats recalculation is triggered when a deposit or withdrawal occurs
  • if possible, frontend is notified to revalidate its strategies cache
    • this one is tricky – it seems strange for trade-executor to "know" about frontend (currently the dependency only goes the other way)
    • perhaps it's sufficient to just shorten the frontend cache TTL to 1 minute (this has already been done)
@kenkunz kenkunz added enhancement New feature or request priority: P3 The lowest priority. A desired nice-to-have improvement, but can be postponed for several releases. size: S A small task that can be completed in less than a day labels Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: P3 The lowest priority. A desired nice-to-have improvement, but can be postponed for several releases. size: S A small task that can be completed in less than a day
Projects
None yet
Development

No branches or pull requests

2 participants