Sell transaction doesn't work properly

[dntexst]

I looked at the code, converted it to javascript and mentioned one problem: sell script doesn’t work properly.
It sends tokens to bonding curve, but doesn’t send fee and doesn’t receive tokens.

I checked example transactions on python and they have the same problem.
Can anyone tell me why this happens? How to fix this?

[geocybercon]

The sell_token() function in your package is inserting the MINIMUM SOL OUTPUT into the instruction, while buy_token() inserts maxiumum lamports... This is a huge fuckup or what? All transactions should be quantified in lamports.

Also, the calculation of minimum yield, ie lamports MINUS slippage, does not subtract, so that is wrong if you're accounting as you should for slippage in the negative. The buy should account for an amount higher than expected purchase to account for positive slippage, and the sell is the opposite, you're accepting a range less than expected to account for droppage during the posting of the transaction.

[akegaviar]

@geocybercon I'm pretty sure you are misunderstanding the slippage mechanics with your claims.

Besides, there are a few things that seem very suspicious about you:

• You are acting pretty suspiciously — you are new account
• Your twitter profile is restricted as of this moment for suspicious activitity
• You open a very weird issue and close it: Unnecessary .exe files included in the package which you all encourage people to run?? Beware! Highly shady! Do not run! #61
• You ask for money in this issue Adding more parameters #60 <— I'm deleting that comment but attaching a screenshot for transparency.

I'm leaving this comment here for transparency. Beware.

[akegaviar]

Hey there, starting from the next week (week of March 10) we'll be rolling out updates to the bot v2 based on the feedback and the reported issues, including updating to the latest libs, better error handling etc. We are already actively working on it.

That'll be in a separate branch.

So for this particular issue, I suggest trying out a new version in a separate branch. We'll let you know.

Also, check out the readme for the roadmap that I just added in the README file for the full overview.