Skip to content

fix: chainlink proposals and pool hooks handling #29

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 11, 2025
Merged

fix: chainlink proposals and pool hooks handling #29

merged 3 commits into from
Apr 11, 2025

Conversation

microHoffman
Copy link
Collaborator

@microHoffman microHoffman commented Apr 11, 2025 •
edited
Loading

  • added new registered chainlink price feeds
  • updated logic for retrieving price feeds (we started supporting up to 4 hops in contracts)
  • added sourceOfFunds to the Proposal class and when creating the proposal

@microHoffman microHoffman self-assigned this Apr 11, 2025
tohrxyz
tohrxyz reviewed Apr 11, 2025
View reviewed changes
@Skanislav Skanislav force-pushed the master branch 2 times, most recently from 5158265 to fef634d Compare April 11, 2025 12:08
@Skanislav Skanislav marked this pull request as ready for review April 11, 2025 17:38
@Skanislav Skanislav self-assigned this Apr 11, 2025
@Skanislav Skanislav merged commit eab54e1 into master Apr 11, 2025
2 checks passed
@Skanislav Skanislav deleted the chainlink-fixes branch April 11, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tohrxyz tohrxyz tohrxyz left review comments

Assignees

@microHoffman microHoffman

@Skanislav Skanislav

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@microHoffman @tohrxyz @Skanislav