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

explicit fee for price feeder #65

Merged
merged 1 commit into from
Mar 18, 2022
Merged

explicit fee for price feeder #65

merged 1 commit into from
Mar 18, 2022

Conversation

dakom
Copy link
Contributor

@dakom dakom commented Mar 3, 2022

fixes a regression in 71d4693#diff-54855b3c211c1c73c2545a70b576ed8395ebb84115d405061d2f4255700e8210R136

see also #64

Does not bump the version number in Dockerfile since I don't have access to publish that

@Vritra4
Copy link
Contributor

Vritra4 commented Mar 3, 2022

actually, after tx removed by #60, previous version will work..
before #60 merged, i think you can use terramoney/pseudo-feeder:bombay

@dakom
Copy link
Contributor Author

dakom commented Mar 3, 2022

Thanks for the heads up. I'll close this PR once #60 is merged and I can confirm price feeder works again 🤞

@octalmage
Copy link
Contributor

#60 has been merged! Are we good to close this @dakom?

@hanjukim hanjukim merged commit 34306d6 into terra-money:main Mar 18, 2022
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.

4 participants