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

Travis ci #27

Closed
wants to merge 4 commits into from
Closed

Travis ci #27

wants to merge 4 commits into from

Conversation

johng
Copy link
Contributor

@johng johng commented Dec 29, 2020

Closes #6

Json data through GET doesn't work correctly
@neocarmack
Copy link
Member

i don’t understand. Obd no longer needs full node, so why re-activate [chainNode] in configure/conf.ini? [chainNode] is already deprecated.

@neocarmack
Copy link
Member

There are deep conflict in this PR.

We will change the way of GET to POST in the next version of obd.

and in the configure of tracker (tracker/config/conf.ini), there is your own chainnode config for your local omnicore instance, which should not be merged into main codebase.

And there are some other points we need to review for this PR.

@johng
Copy link
Contributor Author

johng commented Feb 20, 2021

Closing for now

@johng johng closed this Feb 20, 2021
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.

Run tests on PR
2 participants