fix(doc): review topics on maps and big-maps#81
Merged
EduardoRFS merged 6 commits intorinderknecht@new_jsligofrom Jul 11, 2025
Merged
fix(doc): review topics on maps and big-maps#81EduardoRFS merged 6 commits intorinderknecht@new_jsligofrom
EduardoRFS merged 6 commits intorinderknecht@new_jsligofrom
Conversation
EduardoRFS
reviewed
Jul 7, 2025
EduardoRFS
reviewed
Jul 7, 2025
Comment on lines
+220
to
+221
| To simultaneously update a map and obtain the value of the updated element, use the function `Big_map.get_and_update`. | ||
| This function allows you to extract a value from a big-map for use, as in this example: |
Contributor
There was a problem hiding this comment.
It would be interesting to point out that get_and_update is useful for linear values like tickets.
Contributor
Author
There was a problem hiding this comment.
After some exploration, I'm going to handle this in a future PR because I'd need to update info on handling tickets in test with contracts that use entrypoints instead of a single main function.
EduardoRFS
reviewed
Jul 7, 2025
EduardoRFS
reviewed
Jul 7, 2025
EduardoRFS
reviewed
Jul 7, 2025
Comment on lines
+242
to
+243
| To simultaneously update a map and obtain the value of the updated element, use the function `Map.get_and_update`. | ||
| This function allows you to extract a value from a map for use, as in this example: |
Contributor
There was a problem hiding this comment.
Same comment as above about linearity.
EduardoRFS
reviewed
Jul 7, 2025
timothymcmackin
force-pushed
the
tpm/v2-maps-and-big-maps-new-jsligo
branch
from
1c24791 to
a21c6d4
Compare
EduardoRFS
added a commit
that referenced
this pull request
Nov 26, 2025
* Remove old language-basics topic on maps * Rework topics on maps and big-maps * No longer using semicolons here * JsLIGO uses commas for big-maps * This annotation is not necessary * typo Co-authored-by: Eduardo Rafael <theeduardorfs@gmail.com> --------- Co-authored-by: Eduardo Rafael <theeduardorfs@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebase of #62 on new_jsligo.