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

New market #158

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

New market #158

wants to merge 9 commits into from

Conversation

dotevo
Copy link
Contributor

@dotevo dotevo commented May 14, 2018

Still some work in progress

@yamilrh
Copy link

yamilrh commented Jun 2, 2020

Hola e intentado poner su Trade Zone en mi proyecto y en el ultimo paso me dice Objeto no valido..

@dotevo
Copy link
Contributor Author

dotevo commented Jun 3, 2020

@yamilrh What do you mean?

@yamilrh
Copy link

yamilrh commented Jun 4, 2020

haber lo que intente es usar el mercado que tienen aca, el que mandas para la 17, que puede vender tus naves, el Trade Zone, pero al parecer me falta algo, o sea que revertí lo que pones aca. supongo que este es quitando aquel no?

@dotevo
Copy link
Contributor Author

dotevo commented Jun 5, 2020

This PR is quite old. I'm not sure that is working fine with the newest steemnova code. The previous market has additional zone (market zone). This PR removes that zone because it was some kind of fleet/resource saving. You could send all resources and all fleet to this zone.
Here it is working differently. You create a trade request ex. "5000deu" and this position will be visible on the market only if you have those resources on planet.

You can checkout on this PR and check it.

@mys do you think that this PR is still working fine with the newest steemnova?

@yamilrh
Copy link

yamilrh commented Jun 5, 2020

si lo que intento es implementarlo. pero ni modo, no llego

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

Successfully merging this pull request may close these issues.

None yet

2 participants