sui icon indicating copy to clipboard operation
sui copied to clipboard

Expose RGP (Reference Gas Price) to transaction validation

Open dariorussi opened this issue 2 years ago • 2 comments

We are loading RGP in the per epoch store and use it when checking gas for the transaction. Few todo left in case they matter, please advise.

Testing is light for now (I stole a test from @bmwill) but I plan to add more around the epoch changes aspect and more transaction kind. If people are happy with this as is I can get it in, if we want more tests, those are coming soon

dariorussi avatar Feb 16 '23 01:02 dariorussi

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
explorer ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 17, 2023 at 9:34PM (UTC)
explorer-storybook ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 17, 2023 at 9:34PM (UTC)
frenemies ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 17, 2023 at 9:34PM (UTC)
wallet-adapter ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 17, 2023 at 9:34PM (UTC)

vercel[bot] avatar Feb 16 '23 01:02 vercel[bot]

https://github.com/MystenLabs/sui/pull/8366 adds the entire system object so that would be good and I pick it up from there I guess. I am going to try this change anyway just to make it work and I'll sync with @andll to find out what's best

dariorussi avatar Feb 17 '23 02:02 dariorussi

rebased over @andll PR and this got pretty simple... please take a look

dariorussi avatar Feb 17 '23 23:02 dariorussi

Seems non controversial.

very.... :)

dariorussi avatar Feb 18 '23 00:02 dariorussi