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

[FEATURE] Allow Alert notifications to trigger on a different value than price (volume, liquidation) #315

Open
oksuzali999 opened this issue May 18, 2023 · 7 comments
Labels
enhancement New feature or request

Comments

@oksuzali999
Copy link

First of all, thank you for this beautiful work.
Can an alarm be added to liquidation rates as well as prices? For example, in BTC, can I create a warning with an alarm when there is a liquidation of 250 k up or down? How should I go about doing this.

@oksuzali999 oksuzali999 added the enhancement New feature or request label May 18, 2023
@adeacetis
Copy link
Collaborator

Hello,

It's impossible to set an Alert to something different than the price now. However, I reckon that it would be indeed useful. I am adding this to the feature backlog.

FYI @Tucsky

@adeacetis adeacetis reopened this May 19, 2023
@adeacetis adeacetis added good first issue Good for newcomers help wanted Extra attention is needed and removed help wanted Extra attention is needed labels May 19, 2023
@adeacetis adeacetis changed the title Liquidation Alert [FEATURE] Allow Alert notifications to trigger on a different value than price (volume, liquidation) May 19, 2023
@adeacetis adeacetis removed the good first issue Good for newcomers label May 19, 2023
@lmvdz
Copy link
Contributor

lmvdz commented May 19, 2023

lmvdz@bcf9e15

converted alertService.ts to allow for indicator-based management/storage of alerts.
Need help from @Tucsky to get the value of the indicator instead of prices from aggregator service in the getValue function.

lmvdz@389b603#diff-aed1a9b5af387801328e5252658e7eb3b99ec2b2f75f1a237fedc4a73ce58935R302

@lmvdz
Copy link
Contributor

lmvdz commented May 19, 2023

Looking into this more, and will need more work than just the alertService.ts.

  • displaying the alert "price" line on the chart will have to be changed to use the indicator's axis.
  • the way of alt clicking on the chart to quickly create an alert for price perhaps shouldn't be changed
  • creating alerts for indicators should probably be done through a dialog.
    • one way would be to add it to the actual indicator script

@oksuzali999
Copy link
Author

yes, I realized that I need to concentrate on "alertService.ts". Including "lsell" and "lbuy"... I've been trying to achieve this for a long time. This situation requires experience beyond my coding knowledge and I failed to do so.

@raid5
Copy link

raid5 commented May 19, 2023

I'm glad to see this being added to the feature backlog, I came here to request the same thing! I would LOVE volume alerts.

Also, maybe close #238 as a dupe since this issue has more context now?

@mhakby
Copy link

mhakby commented Jul 21, 2023

Any progress on this? If not, can you tell me when it might happen?

@Tucsky
Copy link
Owner

Tucsky commented Jul 21, 2023

@mhakby I wouldn't get my hopes up unless someone explicitly said that hes working on it @mhakby

alertService mentioned above is exclusively for server side price alerts, alerts are then grouped by indexes (avg price of the coin accross the different contracts) and range of prices for saving performance which is incompatible with any other type of value.

A way I can think of, is for you to fork this Github project and edit the Prices.vue pane (as it is the closest thing to a screener)
while making use of this.$store.dispatch('app/showNotice', ...) and audioService.playOnce(your alert sound name) to get the info while the app is open

Another way would be to expose some notification helper (display a message, play audio notification) function in the indicator script to let you get notified of any event that your indicator would be aware of, this would only work for the active markets on the chart. This is probably the easiest way and less usefull way

Staying available here or on discord if someone wants to give it a go and try to implement it. Also PR is very much appreciated, as always :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants