88mphapp / ng88mph-frontend

2 stars 14 forks source link

Remove use of Blocknative products #87

Open 0xSzeth opened 2 years ago

0xSzeth commented 2 years ago

Per McFly, we'd like to discontinue our use of Blocknative products in our app in favor of Alchemy products, if available. As of right now, we use both the Onboard and Notify products. Further investigation is needed to determine what, if anything, will be an adequate replacement for these two products.

0xSzeth commented 2 years ago

Onboard: Blocknative's onboard product is used to easily connect our app with many different wallet providers. Other apps, which are typically built in react, use a library called web3-react to accomplish this easily in their apps. Here is a forked library that was build for angular: https://www.npmjs.com/package/@mindsorg/web3modal-angular?activeTab=readme

Notify: Blocknative's notify product is used to display fairly generic transaction status updated within our app. As of right now, we've implemented a native notification system but have kept the Notify notifications as a backup (e.g. when a user closes the native notification modal. We can eliminate the use of Notify entirely by extending the functionality of the native notifications and adding a transaction status indicator in the header (similar to Sushiswap).