Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[docs] Update ::call_frames::ContractId to ::contract_id::ContractId in Native Assets Documentation #6141

Merged
merged 3 commits into from
Oct 28, 2024

Conversation

0x0077
Copy link
Contributor

@0x0077 0x0077 commented Jun 19, 2024

Description

The syntax std::call_frames::ContractId has been updated in the latest version of Sway. The new syntax is std::contract_id::ContractId. To ensure the documentation is accurate and consistent with the latest version, this update is necessary.

Reason for Change:

  • The std::call_frames::ContractId syntax has been deprecated or modified in the latest version of Sway.
  • Using the latest std::contract_id::ContractId syntax ensures the documentation is accurate and helps developers avoid using outdated syntax.

Reference

#5867

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

@0x0077 0x0077 requested review from a team as code owners June 19, 2024 06:38
@CLAassistant
Copy link

CLAassistant commented Jun 19, 2024

CLA assistant check
All committers have signed the CLA.

@IGI-111 IGI-111 merged commit 7e499fe into FuelLabs:master Oct 28, 2024
42 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants