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

fix: Contract storage page is still referring to the old cairo syntax #959

Merged
merged 1 commit into from
Nov 22, 2023
Merged

Conversation

bigant88
Copy link
Contributor

@bigant88 bigant88 commented Nov 22, 2023

Description of the Changes

Applied new cairo syntax for Contract storage (maybe someone missed these places)

PR Preview URL

After you push a commit to this PR, a preview is built and a URL to the root of the preview appears in the comment feed.

Paste here the specific URL(s) of the content that this PR addresses.

Check List

  • Changes have been done against dev branch, and PR does not conflict
  • PR title follows the convention: <docs/feat/fix/chore>(optional scope): <description>, e.g: fix: minor typos in code

This change is Reviewable

@bigant88 bigant88 changed the title Fix issue #883: Contract storage page is still referring to the old cairo syntax fix: Contract storage page is still referring to the old cairo syntax Nov 22, 2023
@bigant88
Copy link
Contributor Author

@stoobie : could you help me to fix the build process

@stoobie
Copy link
Collaborator

stoobie commented Nov 22, 2023

@stoobie : could you help me to fix the build process

Hey, @bigant88, thanks for your contribution! Yeah, I noticed that the build doesn't seem to happen with PRs that originate from forks. Until I have time to fix it, I'll create a copy of it, which should generate a build preview.

@stoobie stoobie changed the base branch from dev to bigant88/fix_issue-883 November 22, 2023 16:58
@stoobie stoobie merged commit 97a456e into starknet-io:bigant88/fix_issue-883 Nov 22, 2023
1 of 2 checks passed
stoobie added a commit that referenced this pull request Nov 28, 2023
)

* fix: apply new cairo syntax for Contract storage (#959)

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-storage.adoc

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-storage.adoc

---------

Co-authored-by: Thao Do Van <[email protected]>
stoobie added a commit that referenced this pull request Nov 28, 2023
stoobie added a commit that referenced this pull request Nov 28, 2023
#960) (#975)

* fix: apply new cairo syntax for Contract storage (#959)

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-storage.adoc

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-storage.adoc

---------

Co-authored-by: Thao Do Van <[email protected]>
stoobie added a commit that referenced this pull request Nov 28, 2023
* Clarification and edits in Data availabitily

* Edits and clarifications

* Update components/Starknet/modules/architecture_and_concepts/pages/Network_Architecture/on-chain-data.adoc

* Apply suggestions from code review

* fix: apply new cairo syntax for Contract storage (Previously #959) (#960)

* fix: apply new cairo syntax for Contract storage (#959)

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-storage.adoc

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-storage.adoc

---------

Co-authored-by: Thao Do Van <[email protected]>

---------

Co-authored-by: Thao Do Van <[email protected]>
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