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

Fixed issues to mfast works with Elasticia exchange issue#108 &110 #112

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

Conversation

pcbk
Copy link

@pcbk pcbk commented Nov 4, 2021

No description provided.

@pcbk pcbk changed the title Fixed issues to mfast works with Elasticia exchange Fixed issues to mfast works with Elasticia exchange issue#110-112 Nov 10, 2021
@pcbk pcbk changed the title Fixed issues to mfast works with Elasticia exchange issue#110-112 Fixed issues to mfast works with Elasticia exchange issue#108-110 Nov 10, 2021
Adding testcase for issue
Reverted change, fixed by issue 108
@pcbk pcbk changed the title Fixed issues to mfast works with Elasticia exchange issue#108-110 Fixed issues to mfast works with Elasticia exchange issue#108 &110 Nov 10, 2021
removed unused testcase
@huangminghuang
Copy link
Contributor

This change is not baed on the standard encoding rule and would break the interoperability with other exchanges. If you really need this, I would suggest to use macro to enable it when needed.

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.

2 participants