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(taiko-client): updates for protocol preconfirmation changes #17690

Merged
merged 49 commits into from
Jul 16, 2024
Merged

feat(taiko-client): updates for protocol preconfirmation changes #17690

merged 49 commits into from
Jul 16, 2024

Conversation

davidtaikocha
Copy link
Member

No description provided.

YoGhurt111 and others added 19 commits June 28, 2024 14:50
@YoGhurt111
unit test for pre-confirmation propose
d8b7efc
@YoGhurt111
Merge branch 'preconfirmations' into preconfirmations-taiko-client
b9f8ee8
@YoGhurt111
Merge branch 'preconfirmations' into preconfirmations-taiko-client
0b37fa5
@cyberhorsey
change txbuilder api to rely on proposer to pass in the metaHash, l1S…
706ccf4 
…tateBlockNumber, and timestamp when constructing a block
@cyberhorsey
rm accidental file
d4eca52
@cyberhorsey
wip: proposer server
7951c8a
@cyberhorsey
dependencies for server
1bb3e58
@cyberhorsey
unused struct field
7e4e83a
@cyberhorsey
preconf api as its own service
d798b17
@cyberhorsey
lint
a8680c0
@cyberhorsey
calldata
cc1c759
@cyberhorsey
rf
a856daf
@cyberhorsey
rf
7ae9f96
@YoGhurt111
fix lint
16ddd7a
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
4af886c
@cyberhorsey
rm
7d19b75
@cyberhorsey
Merge branch 'preconfirmations-taiko-client' of github.com:taikoxyz/t…
cc1403a 
…aiko-mono into preconfirmations-taiko-client
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
3f53788
@cyberhorsey @github-actions
forge fmt & update contract layout table
06d70ba
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Jul 2, 2024 •
edited
Loading

feat(taiko-client): updates for protocol preconfirmation changes

Generated at commit: b2516f6b02d3db10496158d3a781d961e2897352

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 2
2
0
8
42
54
Dependencies Critical
High
Medium
Low
Note
Total		 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@YoGhurt111
Copy link
Contributor

After removing require(!_config.checkEOAForCalldataDA || msg.sender != tx.origin, "not EOA"); this check, all unit tests passed.

cyberhorsey and others added 15 commits July 3, 2024 11:08
@cyberhorsey
.
c9b2d94
@cyberhorsey
bindings
80c31b3
@cyberhorsey
logs
17638e4
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
db7d731
@cyberhorsey
fixes to server
a11b788
@cyberhorsey
Merge branch 'preconfirmations-taiko-client' of github.com:taikoxyz/t…
90899e7 
…aiko-mono into preconfirmations-taiko-client
@cyberhorsey
feat(taiko-client): preconfirmations taiko client support contracts (#…
14b50c9 
…17745)
@cyberhorsey
fix(taiko-client): blob fetching using wrong timestamp (#17754)
25f4da7
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
2949acd
@cyberhorsey
calldata tx builder
6e52e4c
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
f29ce18
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
a010224
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
0e7d9ab
@cyberhorsey @github-actions
forge fmt & update contract layout table
b2516f6
@cyberhorsey
Merge branch 'preconfirmations' into preconfirmations-taiko-client
1f0d448
@cyberhorsey cyberhorsey marked this pull request as ready for review July 16, 2024 16:35
@cyberhorsey cyberhorsey merged commit 0b2a987 into preconfirmations Jul 16, 2024
5 of 7 checks passed
@cyberhorsey cyberhorsey deleted the preconfirmations-taiko-client branch July 16, 2024 16:35
@cyberhorsey
Copy link
Contributor

Merging this PR into preconfirmations branch as managing contract changes on one branch and client on another is getting old!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Brechtpd Brechtpd Brechtpd left review comments

@YoGhurt111 YoGhurt111 YoGhurt111 left review comments

@cyberhorsey cyberhorsey cyberhorsey left review comments

Assignees
No one assigned
Labels
area.client
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@davidtaikocha @YoGhurt111 @cyberhorsey @Brechtpd