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

docs!: fix wrong info at variable outputs page #1962

Merged
merged 13 commits into from
Mar 28, 2024

Conversation

Torres-ssf
Copy link
Contributor

@Torres-ssf Torres-ssf commented Mar 27, 2024

This is a simple fix for the wrong information I've added when writing the doc page variable-outputs.

I've also:

  • Rename getAssetId to getMintedAssetId.
  • Move the code snippet related to the variable-outputs page to its test suite file.

Breaking Changes

  • Rename getAssetId to getMintedAssetId

@Torres-ssf Torres-ssf added the docs Requests pertinent to documentation label Mar 27, 2024
@Torres-ssf Torres-ssf self-assigned this Mar 27, 2024
Torres-ssf and others added 3 commits March 28, 2024 08:54
Co-authored-by: Daniel Bate <djbate23@gmail.com>
Co-authored-by: Daniel Bate <djbate23@gmail.com>
@Torres-ssf Torres-ssf changed the title docs: fix wrong info at variable outputs page docs!: fix wrong info at variable outputs page Mar 28, 2024
Co-authored-by: Peter Smith <peter@blueoceancomputing.co.uk>
@fuel-service-user
Copy link
Contributor

fuel-service-user commented Mar 28, 2024

✨ A PR has been created under the rc-1962 tag on fuels-wallet repo.
FuelLabs/fuels-wallet#1208

@Torres-ssf Torres-ssf enabled auto-merge (squash) March 28, 2024 17:57
Copy link
Contributor

Coverage Report:

Lines Branches Functions Statements
81.22%(+0%) 71.56%(+0%) 78.77%(+0%) 81.32%(+0%)
Changed Files:

Coverage values did not change👌.

@Torres-ssf Torres-ssf merged commit 2c605a5 into master Mar 28, 2024
15 checks passed
@Torres-ssf Torres-ssf deleted the st/docs/fix-info-variable-outputs branch March 28, 2024 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Requests pertinent to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants