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

chore: update beta-5 #139

Merged
merged 13 commits into from
Jan 11, 2024
Merged

chore: update beta-5 #139

merged 13 commits into from
Jan 11, 2024

Conversation

pedronauck
Copy link
Contributor

No description provided.

@pedronauck pedronauck marked this pull request as ready for review December 21, 2023 23:09
@@ -4,7 +4,7 @@ FROM node:20-slim AS base

# Expose the ENVs to the env of the container
ENV PORT="${PORT}"
ENV FUEL_PROVIDER_URL="${FUEL_PROVIDER_URL}"
ENV FUEL_PROVIDER_BETA5="${FUEL_PROVIDER_BETA5}"
Copy link
Member

Choose a reason for hiding this comment

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

Why we change the name of the provider here?

Copy link
Contributor

Choose a reason for hiding this comment

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

FUEL_PROVIDER_URL was better IMO to avoid needing to change later.

Comment on lines -141 to -147
regularEl={
<>
Gas used: {formatZeroUnits(tx.gasUsed || '')}
<br />
Gas limit: {formatZeroUnits(tx.gasLimit || '')}
</>
}
Copy link
Member

@luizstacio luizstacio Jan 10, 2024

Choose a reason for hiding this comment

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

Gas limit is available for transaction of type Script, you can find it on the policies array and on the field scriptGasLimit.

Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

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

in transaction screen, can we hide the fee block when it's a mint transaction?

@pedronauck pedronauck changed the base branch from main to beta-5 January 11, 2024 17:42
@pedronauck pedronauck merged commit f35fb47 into beta-5 Jan 11, 2024
9 checks passed
@pedronauck pedronauck deleted the pn/feat/beta-5 branch January 11, 2024 17:52
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.

4 participants