I’d like to hear the views of other council members regarding the funding of exchange work. As I understand, funding exchange listings are frowned upon but this seems different to me as it would be a grin only exchange to support its distribution.
2 Likes
I would be in favor or funding this if they can get a POC working. I don’t personally see funding a community run exchange to be against the spirit of Grin and it’s work that could potentially be re-used or forked by anyone who wants to run a Grin exchange themselves.1
2 Likes
Ok I’ll get a POC up and we can talk funding later.
Or you may use the same withdraw process of the https://gaeapool.com/ .
user clicks on withdraw → put amount → it immediately generates the slatepack → paste that slatepack in the wallet → then our wallet will generate another code → paste that code in the web to confirm, that’s it.
Anyways, I really like how tradeogre managed things for grin. Unfortunately, it’s not open source, I hope we could fund if someone can public the source code for grin exchange.
2 Likes
This one will be open source so we can build it exactly how we want it and it can last forever.
2 Likes
What about integration into basicswap · GitHub SuchWOW!
1 Like
Or maybe this could be a start for you? GitHub - pkariz/grin-testnet-exchange
1 Like
That’d be a DEX. Roadmap is CEX first.
Might try and use the deposit/withdraw code from here.
1 Like
I finished the most minimalistic version of this exchange so we can start trading on this via SQL commands. Unfortunately, I am inbetween leases/ homes for the next couple weeks so I can’t host the server until I have a permanent residence. I will report back here when I have a place to host from.
6 Likes
I have the test server ready to host the exchange for it’s testing period @ https://exchange.grinminer.net
Just waiting to get the files and instructions from @k26dr
5 Likes
Not a CC member but I support CC using funds for market making purposes
2 Likes
Code is here: GitHub - k26dr/grin-exchange
Not tested but throw it up and I can try running code on it
2 Likes
Thanks for the effort, just did a quick look, I will look further into it later.
Quick check: Maybe you should use DECIMAL instead of INT in balaces table
1 Like
Good catch. I pushed a commit to update that: update balance from int to decimal · k26dr/grin-exchange@d4efc23 · GitHub
1 Like