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

Only encode state change metadata if flag is set #884

Merged
merged 1 commit into from
Dec 22, 2023
Merged

Only encode state change metadata if flag is set #884

merged 1 commit into from
Dec 22, 2023

Conversation

lazynina
Copy link
Member

No description provided.

@lazynina
Copy link
Member Author

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@lazynina lazynina force-pushed the state-change-metadata-encoding-flag branch from ed80418 to ed42bec Compare December 21, 2023 01:32
@lazynina lazynina marked this pull request as ready for review December 21, 2023 01:33
@lazynina lazynina requested a review from a team as a code owner December 21, 2023 01:33
@lazynina lazynina force-pushed the state-change-metadata-encoding-flag branch from ed42bec to 91bb624 Compare December 21, 2023 18:24
superzordon
superzordon approved these changes Dec 21, 2023
View reviewed changes
Copy link
Contributor

@superzordon superzordon left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@lazynina lazynina merged commit 7393afd into main Dec 22, 2023
3 checks passed
@lazynina lazynina deleted the state-change-metadata-encoding-flag branch December 22, 2023 01:10
lazynina added a commit that referenced this pull request Dec 22, 2023
@lazynina
Merge main into feature pos (#886)
70c7888 
* Add extra data to max spend and diamond txn construction (#883)

* Only encode state change metadata if flag is set (#884)

Co-authored-by: Lazy Nina <>

---------

Co-authored-by: Lazy Nina <>
AeonSw4n pushed a commit that referenced this pull request Jan 29, 2024
@lazynina @AeonSw4n
Merge main into feature pos (#886)
af66ba4 
* Add extra data to max spend and diamond txn construction (#883)

* Only encode state change metadata if flag is set (#884)

Co-authored-by: Lazy Nina <>

---------

Co-authored-by: Lazy Nina <>
AeonSw4n pushed a commit that referenced this pull request Jan 29, 2024
@lazynina @AeonSw4n
Merge main into feature pos (#886)
d88fff2 
* Add extra data to max spend and diamond txn construction (#883)

* Only encode state change metadata if flag is set (#884)

Co-authored-by: Lazy Nina <>

---------

Co-authored-by: Lazy Nina <>
AeonSw4n pushed a commit that referenced this pull request Jan 29, 2024
@lazynina @AeonSw4n
Merge main into feature pos (#886)
5197f64 
* Add extra data to max spend and diamond txn construction (#883)

* Only encode state change metadata if flag is set (#884)

Co-authored-by: Lazy Nina <>

---------

Co-authored-by: Lazy Nina <>
AeonSw4n pushed a commit that referenced this pull request Jan 29, 2024
@lazynina @AeonSw4n
Merge main into feature pos (#886)
0525ef5 
* Add extra data to max spend and diamond txn construction (#883)

* Only encode state change metadata if flag is set (#884)

Co-authored-by: Lazy Nina <>

---------

Co-authored-by: Lazy Nina <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@superzordon superzordon superzordon approved these changes

@diamondhands0 diamondhands0 Awaiting requested review from diamondhands0

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lazynina @superzordon