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

[DO NOT MERGE] - Triage PR to view Cadence updates following audit #105

Closed
wants to merge 136 commits into from
Closed

[DO NOT MERGE] - Triage PR to view Cadence updates following audit #105

wants to merge 136 commits into from

Conversation

sisyphusSmiling
Copy link
Contributor

⚠️ DO NOT MERGE - Drafting this PR for @nialexsan @bthaile can see a diff from main against early Flow Port integration code

@sisyphusSmiling
Merge pull request #76 from onflow/standup-script
6e5d9c2 
Bridge standup script + Flow Port & Migrationnet prep
@sisyphusSmiling
add EVM serialization updates & FLOW transfer txn
36d7acb
@sisyphusSmiling
Merge pull request #77 from onflow/add-flow-transfer-txn
7cbab5c 
Add EVM serialization updates & FLOW transfer txn
@sisyphusSmiling
fix NFT templates
c9f492f
@sisyphusSmiling
update NFT standard
e41d781
@sisyphusSmiling
add collection & vault configuration scripts
fac0e0f
@sisyphusSmiling
add Flow port supporting Vault summary script
f3538cf
@sisyphusSmiling
add comments to token query
ba7e726
@sisyphusSmiling
add test coverage for vault & collection config queries
45f058f
@sisyphusSmiling
Merge pull request #83 from onflow/fix-templates
efd99b1 
Fix NFT contract templates
@sisyphusSmiling
remove EVMUtils & implementations
34fa385
@sisyphusSmiling
fix get_evm_address_string_from_bytes script
2a60d7e
@sisyphusSmiling
Merge pull request #84 from onflow/remove-evm-utils
a50fdd6 
Remove EVMUtils & implementations
@sisyphusSmiling
fix BridgePermissions.sol
e171010
@sisyphusSmiling
fix FlowEVMBridgeConfig.evmAddressHasTokenHandler
8ecd0a9
@sisyphusSmiling
add revert on collision in FlowEVMBridgeNFTEscrow.initializeEscrow
6c77dc6
@sisyphusSmiling
update returned COA entitlement by FlowEVMBridgeUtils.borrowCOA
412ecdd
@sisyphusSmiling
add input validation in FlowEVMDeploymentRegistry & FlowEVMBridgeConf…
d94f085 
…ig & fix escrow overflow
@sisyphusSmiling
bump compiler v & lock pragma + update bytecode consts
e6cb803
@sisyphusSmiling
remove ownership renunciation from FlowBridgeFactory
0dcfe7c
@sisyphusSmiling
fix FlowEVMBridgeUtils.getEVMOnboardingValues to assert on ERC20
aeef4ea
@sisyphusSmiling
add ICrossVM interface, update bridged ERC20/721 template & fix ERC16…
4481141 
…5 impl
@sisyphusSmiling
update contract events
9dba339
@sisyphusSmiling
fix mismatched behavior & documentation in FlowEVMBridgeUtils
8339633
@sisyphusSmiling
add gasLimit to FlowEVMBridgeConfig in place of hardcoded val
e873aa3
@sisyphusSmiling
add transaction post-conditions where possible
837063e
@sisyphusSmiling
fix hardcoded bridged nft name when resolving NFTCollectionDisplay fr…
f631508 
…om contract
@sisyphusSmiling
add type-level pausing via FlowEVMBridgeConfig & add test case
efc4256
@sisyphusSmiling
fix dict & arr serialization + toUpper bug on numbers
537e0ef
@sisyphusSmiling
updates for best practice & docs recommendations post-audit
a71a74b
sisyphusSmiling and others added 29 commits September 2, 2024 18:59
@sisyphusSmiling
update metadata setting & routes for bridged assets
6362f76
@sisyphusSmiling
add coverage for bridged vault views
ca36c6a
@sisyphusSmiling
update templates & template code chunk args
80f8dbe
@sisyphusSmiling
add migrationmainnet to flow.json
11bae2d
@sisyphusSmiling
add new contracts to bridge standup script
6c87ab9
@sisyphusSmiling
add bridged metadata setter args
d81748d
@sisyphusSmiling
update transaction comments
1f6e1b8
@sisyphusSmiling
update transaction comment
3f3ae9f
@sisyphusSmiling @joshuahannan
Update cadence/contracts/bridge/FlowEVMBridgeResolver.cdc
e1980f3 
Co-authored-by: Joshua Hannan <[email protected]>
@sisyphusSmiling
remove previewnet templates & args
2acad5b
@sisyphusSmiling
add util methods to FlowEVMBridgeResolver
75ad26b
@sisyphusSmiling
update set_bridged_nft_display_view to use new util methods
b678363
@sisyphusSmiling
correct templates and committing json args
350fa14
@sisyphusSmiling
Merge pull request #116 from onflow/update-templates
ce09dd2 
Update bridged metadata resolution & templates
@sisyphusSmiling
add the ability to update symbols in bridged ERC20/721 contracts
8902cab
@sisyphusSmiling
Merge pull request #118 from onflow/enable-symbol-updates
ca95d45 
Add the ability to update symbols in bridged ERC20/721 contracts
@sisyphusSmiling
add type assertion to onboarding Vault
fa5f613
@sisyphusSmiling
Merge pull request #119 from onflow/add-onboarding-type-assertion
7d4a2ff 
Add type assertion to onboarding Vault
@sisyphusSmiling
update type identifiers in metadata view args
11d97da
@sisyphusSmiling
fix incorrect alias in emulator bridged NFT template
baae22f
@sisyphusSmiling
remove previewnet & crescendo networks from flow.json
979f556
@sisyphusSmiling
add deployment addresses to README
e37e779
@sisyphusSmiling
update ci commands for new CLI version
0758fcb
@sisyphusSmiling
add deployer supporting transaction & script
baf47a1
@sisyphusSmiling
replace current dependency management w/ dependency manager
5e5f588
@sisyphusSmiling
add imports/ to .gitignore
a6c91d9
@sisyphusSmiling
remove emulator deployment from flow.json
e5aa113
@sisyphusSmiling
fix bridge interface error messages
66629f2
@sisyphusSmiling
Merge pull request #120 from onflow/post-crescendo-updates
8c83149 
Post-Crescendo updates
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
Chore
Projects
FlowEVM Bridge
Status: Done
🌊 Flow 4D
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sisyphusSmiling @codecov-commenter @turbolent