Morph21 / MercuryTrade-Community-Fork

An overlay application for Path of Exile(Tracking, Trading, Chat, etc)
MIT License
140 stars 24 forks source link

[FEATURE REQUEST] - Deduplicate automatic trade whispers #46

Open rmelcer opened 3 years ago

rmelcer commented 3 years ago

Is your feature request related to a problem? Please describe. When a MercuryTrade user uses PoE Overlay's shortcuts (/hideout, /kick), MT will re-whisper old trade messages.

Ex:

Other minor problem:

  1. When MT starts or restarts, if your clipboard contains an old trade message, MT will resend it.
  2. When the user refreshes their item search page, they may lose track of which items they have already messaged. In this case, if they have already messaged for an item, MT will ping the seller again.

Describe the solution you'd like I'd like MT to check the clipboard's contents against its history of previous trades, and skip resending the whisper if it was already sent recently (last 5-10 messages) Describe alternatives you've considered I tried to make this myself and it seemed to be working (#45), but after clearing my history, it broke again. When I dug into it more, it seems that MT only saves the message (Hi I'd like to buy your 2 chaos for my 10 alchemy) without including the recipient's name. This makes it impossible to de-duplicate across similar trades to different users - for example with multiple currency trades. Additional context Add any other context or screenshots about the feature request here.

Morph21 commented 3 years ago

you know that you can use /hideout macro in MT also? Just bind the shortcut. And here it's basically poe overlay fault for returning clipboard to the state which it was before. So to prevent this from happen MT would need to scan entire Client.txt file after each clipboard message which would be time consuming. Also MT is not saving history like in file or something. It's just in memory of running application so when you restart MT th history is lost