## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR focuses on updating contract initializer parameters from arrays to objects for better clarity and consistency.
### Detailed summary
- Updated contract initializer params from arrays to objects for improved readability
- Refactored multiple files to use object-based contract params
- Added normalization function for function parameters in zkDeployContract file
> The following files were skipped due to too many changes: `packages/thirdweb/src/contract/deployment/zksync/zkDeployContract.ts`, `packages/thirdweb/src/utils/abi/normalizeFunctionParams.ts`, `packages/thirdweb/src/extensions/prebuilts/deploy-published.test.ts`, `packages/thirdweb/test/src/chains.ts`, `packages/thirdweb/src/contract/deployment/zksync/zkDeployDeterministic.ts`, `packages/thirdweb/src/extensions/prebuilts/deploy-marketplace.ts`, `packages/thirdweb/src/utils/any-evm/compute-published-contract-address.ts`, `packages/thirdweb/src/utils/any-evm/compute-published-contract-deploy-info.ts`, `packages/thirdweb/src/contract/deployment/deploy-deterministic.test.ts`, `packages/thirdweb/src/extensions/prebuilts/get-required-transactions.test.ts`, `packages/thirdweb/src/extensions/prebuilts/get-required-transactions.ts`, `packages/thirdweb/src/extensions/prebuilts/deploy-modular-core.test.ts`, `packages/thirdweb/src/contract/deployment/utils/infra.ts`, `packages/thirdweb/src/extensions/prebuilts/deploy-modular.ts`, `packages/thirdweb/src/contract/deployment/utils/bootstrap.ts`, `packages/thirdweb/src/extensions/prebuilts/deploy-published.ts`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR focuses on adding tests for ERC20 token `transferFrom` function with different `amount` and `amountWei` parameters.
### Detailed summary
- Added tests for `transferFrom` function with `amount` parameter
- Added tests for `transferFrom` function with `amountWei` parameter
- Utilized test utilities and prebuilt functions
- Checked balance after transfer for correctness
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to add test cases for the `getApprovalForTransaction` function in the ERC20 extension.
### Detailed summary
- Added test cases for `getApprovalForTransaction` function
- Imported necessary modules for testing
- Defined test parameters such as chain, client, and accounts
- Tested approval transaction scenarios
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR focuses on adding tracking events for various payment methods and transactions across different screens.
### Detailed summary
- Added a new function `trackPayEvent` for tracking payment events
- Added tracking events for different payment methods like buy, direct payment, and swap
- Updated multiple files to include tracking events for specific user actions
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
### TL;DR
Enhanced the WalletAddress component to handle undefined addresses and improved address display.
### What changed?
- Added support for undefined addresses by defaulting to ZERO_ADDRESS
- Updated address processing to use the new default address
- Made the WalletAvatar conditional on the presence of an address
- Imported ZERO_ADDRESS from the thirdweb library
### How to test?
1. Render the WalletAddress component with an undefined address
2. Verify that it displays the ZERO_ADDRESS
3. Test with valid addresses to ensure proper shortening and display
4. Check that the WalletAvatar only appears when an address is provided
### Why make this change?
This change improves the robustness of the WalletAddress component by gracefully handling undefined addresses. It prevents potential errors and provides a consistent display even when address data is missing or not yet loaded.
---
<!-- start pr-codex -->
## PR-Codex overview
This PR updates NFT transformation functions in `alchemy.ts` and `moralis.ts` to use `download` method with `thirdwebClient` for fetching metadata.
### Detailed summary
- Updated NFT transformation functions to use `download` method with `thirdwebClient` for fetching metadata
- Replaced `StorageSingleton.downloadJSON` with `download` method in both files
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to refactor the login process by simplifying the `login` function and updating the login logic in the login page component.
### Detailed summary
- Simplified `login` function in `auth.ts`
- Updated login logic in `login/page.tsx` to use the simplified `login` function and navigate to wallet page after successful login
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
<img width="1143" alt="Screenshot 2024-09-10 at 2 30 34 PM" src="https://github.com/user-attachments/assets/c3848dce-0bb2-4bd7-ac8a-300f1757a207">
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to add social profile functionality and API documentation to the app.
### Detailed summary
- Added "Social" link to nav bar
- Implemented `Badge` component with variants
- Created `SocialProfiles` component to display user profiles
- Added API documentation for social profiles
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to add support for LINE authentication in various parts of the application.
### Detailed summary
- Added LINE authentication option in multiple files
- Updated constants and icons for LINE
- Integrated LINE in authentication flows
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
before:
<img width="1942" alt="image" src="https://github.com/user-attachments/assets/8ac59faa-f81c-4670-98e5-f750ce68e5d6">
after:
<img width="1819" alt="image" src="https://github.com/user-attachments/assets/c8c015eb-528b-44a7-8cd5-478ca3ca907e">
<!-- start pr-codex -->
---
## PR-Codex overview
This PR adds social API domain support, new UI components, and updates dependencies.
### Detailed summary
- Added `THIRDWEB_SOCIAL_API_DOMAIN` in `urls.ts`
- Added `WalletAddress` component
- Updated UI components in various files
- Added new dependencies like `@radix-ui/react-avatar` and `@radix-ui/react-hover-card`
> The following files were skipped due to too many changes: `apps/dashboard/src/components/wallets/ConnectWalletMiniPlayground/MiniPlayground.tsx`, `apps/dashboard/src/contract-ui/tabs/analytics/page.tsx`, `apps/dashboard/src/components/contract-components/contract-deploy-form/split-fieldset.tsx`, `apps/dashboard/src/components/contract-components/contract-deploy-form/custom-contract.tsx`, `apps/dashboard/src/contract-ui/tabs/manage/components/InstalledModulesTable.tsx`, `apps/dashboard/src/components/contract-components/contract-table-v2/index.tsx`, `apps/dashboard/src/components/pay/PayAnalytics/components/PaymentHistory.tsx`, `apps/dashboard/src/contract-ui/tabs/overview/components/MarketplaceDetails.tsx`, `apps/dashboard/src/contract-ui/tabs/nfts/components/table.tsx`, `apps/dashboard/src/pages/[chain_id]/[...paths].tsx`, `apps/dashboard/src/components/embedded-wallets/Users/index.tsx`, `apps/dashboard/src/components/engine/overview/engine-overview.tsx`, `apps/dashboard/src/components/engine/permissions/access-tokens-table.tsx`, `apps/dashboard/src/contract-ui/tabs/shared-components/marketplace-table.tsx`, `apps/dashboard/src/pages/dashboard/connect/analytics.tsx`, `apps/dashboard/src/app/(dashboard)/styleguide/page.tsx`, `apps/dashboard/src/contract-ui/tabs/account-permissions/components/account-signer.tsx`, `apps/dashboard/src/components/engine/engine-instances-table.tsx`, `apps/dashboard/src/components/engine/overview/backend-wallets-table.tsx`, `apps/dashboard/src/components/engine/overview/transactions-table.tsx`, `apps/dashboard/src/components/pay/PayAnalytics/components/PayCustomersTable.tsx`, `apps/dashboard/src/@/components/ui/hover-card.tsx`, `apps/dashboard/src/@/components/ui/avatar.tsx`, `apps/dashboard/src/@/components/blocks/wallet-address.tsx`, `pnpm-lock.yaml`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR removes the `z-index` property from the `ScrollShadow` component in the dashboard UI to prevent stacking issues.
### Detailed summary
- Removed `z-index` property from `ScrollShadow` component in `ScrollShadow.tsx`
- Adjusted styles in `ScrollShadow.module.css` to remove `z-index` property
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR updates API key handling and smart wallet components.
### Detailed summary
- Added `isOpChainId` function for chain validation
- Modified `getAuthToken` and `getAPIKey` functions
- Updated `SponsorshipPolicies` and `SmartWallets` components
- Introduced `AccountAbstractionPage` for account abstraction details
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
Users are reporting the inability to login on the modal as a "bug" and it's causing confusion while trying to reproduce actual bugs. This PR allows connecting on the playground modal as is the expected behavior.
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to update the `navLinks.ts` file and refactor the `RightSection.tsx` component in the `playground-web` app.
### Detailed summary
- Updated the name from "in-App wallet" to "In-App wallet" in `navLinks.ts`
- Refactored the `RightSection` component in `RightSection.tsx`
- Removed the `useDisconnect` hook and refactored the disconnect logic in `RightSection.tsx`
- Improved conditional rendering for the modal in `RightSection.tsx`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR fixes an error related to ERC721 delayed reveal simulation.
### Detailed summary
- Fixed error in ERC721 delayed reveal simulation
- Added tests for `reveal` function
- Added `createAndReveal` function for batch creation and reveal
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to update the StepsCard component styling and add a new hook `useDashboardRouter` for navigation in the GetStartedWithContractsDeploy component.
### Detailed summary
- Updated styling in `StepsCard` component
- Added `useDashboardRouter` hook for navigation in `GetStartedWithContractsDeploy` component
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
<!-- start pr-codex -->
## PR-Codex overview
The focus of this PR is to update the `SettingsMetadata` component in `metadata.tsx` to resolve image URIs using the `resolveScheme` function from `thirdweb/storage`.
### Detailed summary
- Added import for `resolveScheme` from `thirdweb/storage`
- Resolved image URIs using `resolveScheme` function
- Updated the dependencies of the `useMemo` hook to include `contract.client`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR updates the import in `useProfiles.ts` to use `useProfiles` instead of `use`.
### Detailed summary
- Updated import from `use` to `useProfiles` in `useProfiles.ts`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to refactor imports in the `publish.tsx` file in the `dashboard` component.
### Detailed summary
- Added imports for `Badge`, `Button`, and `Select` components
- Updated imports for `usePathname` and `useSearchParams`
- Removed duplicate imports for `Select` related components
- Updated import paths for certain components
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR removes the `favicon.ico` file from the `wallet-ui` app.
### Detailed summary
- Deleted the `favicon.ico` file from the `wallet-ui` app.
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
<!-- start pr-codex -->
## PR-Codex overview
This PR updates contract components and SDK with default modules support.
### Detailed summary
- Added `defaultModules` support in SDK and contract components
- Updated contract card to display modules
- Added transfer validator field in royalty fieldset
- Improved contract publishing information display
> The following files were skipped due to too many changes: `apps/dashboard/src/components/contract-components/shared/published-by.tsx`, `apps/dashboard/src/components/contract-components/contract-deploy-form/custom-contract.tsx`, `apps/dashboard/src/components/explore/contract-card/index.tsx`, `apps/dashboard/src/components/pages/publish.tsx`, `apps/dashboard/src/data/explore.ts`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR focuses on restructuring ecosystem-related components and pages in the dashboard.
### Detailed summary
- Deleted `redirects.js`.
- Updated page components for ecosystem permissions and creation.
- Added new components for ecosystem layout and header.
- Modified functions for fetching ecosystem data and ecosystem selection.
> The following files were skipped due to too many changes: `apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/components/client/ecosystem-header.client.tsx`, `apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/EcosystemLandingPage.tsx`, `apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/page.tsx`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to simplify the layout component in the contracts section by removing unnecessary elements and only rendering the children components.
### Detailed summary
- Removed sidebar and mobile sidebar components
- Simplified layout to only render children components
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to add a `data-theme` attribute to the `Card` component in `contact-us.tsx`.
### Detailed summary
- Added `data-theme="light"` attribute to the `Card` component.
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
2024-09-09 22:37:42 +00:00
Jonas DanielsGitHubgithub-actions[bot] <github-actions[bot]@users.noreply.github.com>
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR introduces a new `SettingsNotificationsPage` component to handle notification settings in the dashboard.
### Detailed summary
- Added `SettingsNotificationsPage` component to manage notification settings
- Refactored `Page` component to use `SettingsNotificationsPage`
- Updated UI for notification settings with switches and labels
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Add another storage slot for proxy resolution:
keccak256("org.zeppelinos.proxy.implementation")
ref: https://basescan.org/address/0x833589fCD6eDb6E08f4c7C32D4f71b54bdA02913#code
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to enhance proxy resolution in the `thirdweb` package by adding more storage slots for implementation addresses.
### Detailed summary
- Added more storage slots for proxy resolution in `resolveImplementation.test.ts`
- Updated global setup in `globalSetup` function in `globalSetup.ts`
- Added base chain definitions and RPC URL in `chains.ts`
- Improved implementation address extraction in `resolveImplementation.ts`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
Sets an idempotency key to Engine that prevents duplicate transactions _today_ (UTC).
<!-- start pr-codex -->
---
## PR-Codex overview
This PR updates error message spelling and adds idempotency key to prevent duplicate claims in the testnet faucet API.
### Detailed summary
- Fixed spelling error in error message
- Added idempotency key to prevent duplicate claims
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to export additional social functions and types in the `thirdweb` package.
### Detailed summary
- Exported `SocialProfiles`, `EnsProfile`, `FarcasterProfile`, and `LensProfile` from `social/types.js`
- Added paths for social types in `package.json` under `social` section
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to add a new test in `get-events.test.ts` for getting individual events with filters.
### Detailed summary
- Added a new test for getting individual events with filters in `get-events.test.ts`
- Removed a `console.log` statement in `coinbaseMobileSDK.ts`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
The focus of this PR is to introduce a new component `Progress` using `@radix-ui/react-progress`, update styles, add a new layout component, and improve import handling.
### Detailed summary
- Added a new `Progress` component using `@radix-ui/react-progress`
- Updated styles for various components
- Introduced a new layout component for team/project contracts
- Improved import handling with `useDashboardRouter` usage
> The following files were skipped due to too many changes: `apps/dashboard/src/components/dashboard/StepsCard.stories.tsx`, `apps/dashboard/src/@/components/ui/select.tsx`, `apps/dashboard/src/app/team/[team_slug]/[project_slug]/contracts/_components/GetStartedWithContractsDeploy.tsx`, `apps/dashboard/src/components/dashboard/StepsCard.tsx`, `apps/dashboard/src/pages/dashboard/contracts/deploy.tsx`
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
joaquim-verges
kien-ngo
gregfromstl
jnsdls
Sergiy DybskiyandGitHub
Winston YeoandGitHub
MananTank
Joe-ThirdwebandGitHub
James Harrison SomtoandGitHub
kumaryash90
arcoraven
MykandGitHub