Skip to content

docs: align storage reference comments in translations#902

Merged
XdpCs merged 2 commits into
AmazingAng:mainfrom
yuyua9:fix/storage-reference-es-pt-yuyua9
May 18, 2026
Merged

docs: align storage reference comments in translations#902
XdpCs merged 2 commits into
AmazingAng:mainfrom
yuyua9:fix/storage-reference-es-pt-yuyua9

Conversation

@yuyua9

@yuyua9 yuyua9 commented May 18, 2026

Copy link
Copy Markdown
Contributor

Summary

This applies the storage-reference wording fix from #900 to the Spanish and Brazilian Portuguese translations of 06_ArrayAndStruct.

Student storage _student = student creates a storage reference, not a copy. The existing Spanish and pt-br examples still described it as a copy, which can mislead readers about Solidity storage semantics.

Changes

  • Update the Spanish .sol and README snippets to say reference instead of copy.
  • Update the Brazilian Portuguese README snippet to say reference instead of copy.
  • Restore the missing _student storage reference declaration in the Brazilian Portuguese .sol snippet before it is used.

Validation

  • Verified the translated snippets no longer contain the old copy wording.
  • Verified the pt-br Solidity sample declares _student before assigning fields.
  • Ran git diff --check.

Related: #898, #900

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Spanish and Brazilian Portuguese translations of the 06_ArrayAndStruct lesson to correctly describe Student storage _student = student as creating a storage reference (not a copy), aligning with the fix previously applied in #900.

Changes:

  • Update ES + pt-br README snippets to say “reference” instead of “copy” for the storage assignment.
  • Update ES .sol comment to match the correct storage-reference semantics.
  • Update pt-br .sol to reintroduce the _student storage reference declaration before _student is used.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
Languages/pt-br/06_ArrayAndStruct/readme.md Fixes wording to describe a storage reference instead of a copy.
Languages/pt-br/06_ArrayAndStruct/ArrayAndStruct.sol Updates the method-1 example line to declare a storage reference and fixes the misleading “copy” wording.
Languages/es/06_ArreglosyEstructuras_es/readme.md Fixes wording to describe a reference instead of a copy.
Languages/es/06_ArreglosyEstructuras_es/ArrayAndStruct.sol Fixes wording to describe a reference instead of a copy.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Languages/pt-br/06_ArrayAndStruct/ArrayAndStruct.sol
@XdpCs XdpCs merged commit 8019f6b into AmazingAng:main May 18, 2026
1 check failed
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.

3 participants