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.

Sign up for GitHub

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

Jump to bottom

feat: ✨ various improvements on move and copy statements #822

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

feat: ✨ various improvements on move and copy statements #822

wants to merge 7 commits into from

Conversation

matteo-cristino
Copy link
Collaborator

No description provided.

@matteo-cristino matteo-cristino linked an issue Mar 5, 2024 that may be closed by this pull request
move '' from '' to '' des not follow move statement logic #818
Open
@jaromil
Copy link
Member

jaromil commented Mar 9, 2024

Perhaps we can also pack a new statement into this PR:

When I create the random 'ramdom_jti'
When I write the string '' in 'jti'
When I append the 'url64' of 'ramdom_jti' to 'jti'

We can have a way to copy and move as url64 and other encodings inside a string dictionary:

When I move the `random_jti` as `url64` in `jti`

@matteo-cristino matteo-cristino linked an issue Mar 11, 2024 that may be closed by this pull request
Move object with CODEC change #618
Open
@matteo-cristino
fix: 🐛 use correct keyword 'in' when moving stuff inside an object
43bf1e4
@matteo-cristino
refactor: ♻️ move some move and copy statement to zencode_table to av…
45fb7b5 
…oid code duplication
@matteo-cristino
fix: 🐛 statements to copy contents of objects in tables now accept al…
14ed604 
…so arrays and open schemas as destination

Improve also checks on not averwriting existing variables
@matteo-cristino
feat: ✨ add correct statement to move element from object to variable
0436fcd 
If destionation element already exist a custom error is thrown to let the user know the right statement to use since till now this statement was use to move stuff in a object
@matteo-cristino
feat: ✨ add statement to copy element from object in new_object
9af05e2 
In this way we have the same statements both for move and copy
@matteo-cristino
feat(table): ✨ statement to move or copy in a table with a certain en…
1318042 
…coding

This statements support also schema as encoding
@matteo-cristino
feat(when): ✨ add statement to move or copy to an element with a cert…
ae57c94 
…ain encoding

remove some debug leftover and move the fucntion to apply a new encoding in zencode_data.lua
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

move '' from '' to '' des not follow move statement logic Move object with CODEC change
2 participants
@matteo-cristino @jaromil