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.

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

[Dashboard] Feature: UI for crosschain modules #5399

Merged
merged 1 commit into from
Feb 14, 2025
Merged

Conversation

GWSzeto
Copy link
Contributor

@GWSzeto GWSzeto commented Nov 12, 2024

https://linear.app/thirdweb/project/[modular-contracts]-op-interoperability-module-87740e1390a9/overview


PR-Codex overview

This PR introduces new functionalities and improvements related to cross-chain deployments, event handling, and contract deployment mechanisms in the thirdweb package.

Detailed summary

  • Added getDeployedCloneFactoryContract export.
  • Introduced moduleInstalledEvent export.
  • Enhanced _utils/getContractPageSidebarLinks.ts with a new "Cross Chain (Beta)" link.
  • Updated keccak256 result handling in namehash.ts.
  • Created NoCrossChainPrompt component for user notifications.
  • Added optional nonce in installPublishedModule.
  • Implemented cross-chain deployment logic in deploy-via-autofactory.ts.
  • Updated event handling for ProxyDeployedV2.
  • Enhanced deployContractfromDeployMetadata to support cross-chain options.
  • Refactored DataTable to manage cross-chain deployment status.
  • Improved error handling and local storage management for cross-chain data.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

changeset-bot bot commented Nov 12, 2024

⚠️ No Changeset found

Latest commit: 557d3e3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Nov 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 6:49pm
login ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 6:49pm
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 6:49pm
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 6:49pm
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2025 6:49pm

Copy link

graphite-app bot commented Nov 12, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@github-actions github-actions bot added the Dashboard Involves changes to the Dashboard. label Nov 12, 2024
Copy link

codecov bot commented Nov 12, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 10 lines in your changes missing coverage. Please review.

Project coverage is 56.86%. Comparing base (06d6504) to head (557d3e3).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
.../src/contract/deployment/deploy-via-autofactory.ts 72.97% 10 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5399   +/-   ##
=======================================
  Coverage   56.85%   56.86%           
=======================================
  Files        1166     1166           
  Lines       64526    64576   +50     
  Branches     5223     5238   +15     
=======================================
+ Hits        36686    36720   +34     
- Misses      27115    27130   +15     
- Partials      725      726    +1     
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from 06d6504
packages 55.07% <83.33%> (+0.01%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
...xtensions/modules/common/installPublishedModule.ts 100.00% <100.00%> (ø)
...rdweb/src/extensions/prebuilts/deploy-published.ts 85.76% <100.00%> (+0.74%) ⬆️
packages/thirdweb/src/utils/ens/namehash.ts 91.30% <100.00%> (+1.30%) ⬆️
.../src/contract/deployment/deploy-via-autofactory.ts 76.10% <72.97%> (-0.09%) ⬇️

... and 5 files with indirect coverage changes

Copy link
Contributor

github-actions bot commented Nov 12, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 46.3 KB (0%) 926 ms (0%) 3.2 s (+1.68% 🔺) 4.2 s
thirdweb (cjs) 121.89 KB (0%) 2.5 s (0%) 5.9 s (+1.55% 🔺) 8.3 s
thirdweb (minimal + tree-shaking) 5.6 KB (0%) 112 ms (0%) 292 ms (-29.2% 🔽) 404 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 119 ms (+11.18% 🔺) 129 ms
thirdweb/react (minimal + tree-shaking) 19.29 KB (0%) 386 ms (0%) 571 ms (+7.52% 🔺) 957 ms

@gregfromstl
Copy link
Member

@GWSzeto could you reference a linear ticket in the description

@gregfromstl gregfromstl changed the title UI for crosschain modules [Dashboard] Feature: UI for crosschain modules Nov 13, 2024
https://linear.app/thirdweb/project/[modular-contracts]-op-interoperability-module-87740e1390a9/overview

<!-- start pr-codex -->

---

## PR-Codex overview
This PR introduces new features and improvements related to cross-chain contract deployment in the `thirdweb` framework, enhancing functionality and usability for developers.

### Detailed summary
- Added `getDeployedCloneFactoryContract` export in `contract.ts`.
- Introduced `moduleInstalledEvent` export in `modules.ts`.
- Updated TypeScript documentation links.
- Enhanced sidebar links in `getContractPageSidebarLinks.ts`.
- Improved `namehash.ts` with typed return value.
- Created `NoCrossChainPrompt` component for user feedback.
- Added `nonce` and `initializeData` options in `installPublishedModule.ts`.
- Updated event handling for `ProxyDeployedV2`.
- Implemented cross-chain deployment logic in various files.
- Updated tests to cover new cross-chain functionality.
- Refactored contract deployment methods to support new parameters.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dashboard Involves changes to the Dashboard. merge-queue Adds the pull request to Graphite's merge queue. packages SDK Involves changes to the thirdweb SDK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants