Skip to content

Switch writes to emit SpendValueInGasUnits as well as SpendValue in n… - #726

Open
russell-stern wants to merge 1 commit into
mainfrom
non_evm_metering
Open

Switch writes to emit SpendValueInGasUnits as well as SpendValue in n…#726
russell-stern wants to merge 1 commit into
mainfrom
non_evm_metering

Conversation

@russell-stern

Copy link
Copy Markdown

…ative tokens

@github-actions

Copy link
Copy Markdown
Contributor

👋 russell-stern, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@cl-sonarqube-production

Copy link
Copy Markdown

func GetResponseMetadataWriteReport(fee *big.Float, chainSelector uint64) capabilities.ResponseMetadata {
// GetResponseMetadataWriteReport returns billing ResponseMetadata for a completed write-report
// submission.
func GetResponseMetadataWriteReport(feeInEth *big.Float, feeInWei *big.Int, chainSelector uint64) capabilities.ResponseMetadata {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For all GetResponseMetadataWriteReport, pass feeInWei and convert to legacy format in this method

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

Successfully merging this pull request may close these issues.

2 participants