dense-analysis / dense-analysis/pricewarp

Allow arbitrary sorting of price alerts

Open
#23 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Go
Stars
8
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Storing a `position` field for price alerts, which could be `NULL`, we should support sorting price alerts by `position`. `NULL` values should be sorted to the button, and the alerts should otherwise appear in the insertion order they appear in currently. (Order by position, nulls last, then order by `id`.)

Implementation detail: Updating the position of elements will require updating all other alerts for a user to shift the values and database transactions.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.