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

feat: added a dynamic smart contract(Price Feed) #152

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

Conversation

aviral727
Copy link

@aviral727 aviral727 commented Oct 26, 2022

Description

Created a smart contract which uses an oracle(chainlink price feed) for getting real time price of eth/usd

Fixes #9

Dependencies

chainlink contracts v0.8

Future Improvements

Further the contract can be made more intuitive and optimizations can be done

Mentions

NA

Screenshots of relevant screens

image

Developer's checklist

  • My PR follows the style guidelines of this project
  • I have performed a self-check on my work

If changes are made in the code:

  • I have followed the coding guidelines
  • My changes in code generate no new warnings
  • [] My changes are breaking another fix/feature of the project
  • [] I have added test cases to show that my feature works
  • I have added relevant screenshots in my PR

@github-actions
Copy link
Contributor

Thank you for opening your first pull request in our repository, one of our maintainers will get in touch with you soon.

DO NOT FORGET TO ADD METAMASK WALLET ADDRESS IN THE AUTH FORM
We at Metafy have some special rewards and swags for the top contributors in general repos and every contributor in web3 repos
For being eligible for that, please authenticate yourself using this link:
https://metafy.social/signup

Rewards : T-shirts & NFT ( for blockchain contributions )

Best of luck on your Contributions
For further queries you can join our discord for support : https://discord.gg/W6PmEjjQsA

@github-actions
Copy link
Contributor

PR is not linked to any issue, please make the corresponding changes in the body.

@github-actions github-actions bot added PR:Ready-to-Review hacktoberfest-accepted Hacktoberfest Accepted Pull Requests and removed PR:No-Issue labels Oct 26, 2022
@bartick
Copy link
Member

bartick commented Oct 30, 2022

Not quiet sure how is this dynamic. Would have preferred if you are sending the price from the connected app.
Please explain your solution. Also please remove your hardcoded contract address and add a comment there

@advaita-saha please look into it

@aviral727
Copy link
Author

@bartick I thought I just had to get data from an oracle for a dynamic contract. I may have misunderstood, any suggestions how I can rectify it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Hacktoberfest Accepted Pull Requests PR:Ready-to-Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dynamic smart contract
3 participants