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

Store every Bond ROI #4

Open
Zayen-X opened this issue Jul 9, 2021 · 3 comments
Open

Store every Bond ROI #4

Zayen-X opened this issue Jul 9, 2021 · 3 comments

Comments

@Zayen-X
Copy link

Zayen-X commented Jul 9, 2021

Request from @jawz

Quote:

I believe having them show on front end can allow ppl to just get a CTA

Open questions:

  • Is TheGraph the best place for this? We would have to update the graph everytime we release a bond.
@Zayen-X
Copy link
Author

Zayen-X commented Jul 9, 2021

@drondin lets talk about if this is the right thing to do.

@drondin
Copy link
Contributor

drondin commented Jul 9, 2021

hmmm we might be able to do this using this event (only available in latest bond versions, not in old ones)

    event BondCreated( uint deposit, uint indexed payout, uint indexed expires, uint indexed priceInUSD );

@Zayen-X
Copy link
Author

Zayen-X commented Jul 9, 2021

i think that makes sense the new ones are fine going forward.

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

No branches or pull requests

2 participants