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

Optimism refresh #613

Closed
wants to merge 2 commits into from
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Optimism.io refresh
Fixes #609

Update the bridge application status and configurations to reflect the completion of the Optimism.io refresh.

* **README.md**: Remove the note about the bridge application being a work in progress and waiting for more libraries to support Wagmi v2.
* **.eslintrc.cjs**: Turn on the rule 'react-refresh/only-export-components' and set it to 'warn'.
* **package.json**: Update dependencies and scripts to reflect the completion of the refresh, including adding 'vite-plugin-react-refresh' and a new 'refresh' script.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/ethereum-optimism/ecosystem/issues/609?shareId=XXXX-XXXX-XXXX-XXXX).
Abuchtela committed Jan 3, 2025
commit 35add887f85b14557010c8d1e3a3be1a4895e462
2 changes: 1 addition & 1 deletion apps/bridge-app/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -21,6 +21,6 @@ module.exports = {
'error',
{ "varsIgnorePattern": "_", "argsIgnorePattern": "_" }
],
'react-refresh/only-export-components': ['off']
'react-refresh/only-export-components': ['warn']
},
}
2 changes: 0 additions & 2 deletions apps/bridge-app/README.md
Original file line number Diff line number Diff line change
@@ -2,8 +2,6 @@

This is an example bridge application demonstrating how to bridge ETH & ERC20 tokens to any OP stack chain. We currently deploy this on every merge to the `main` branch. You can see what it looks like [here](https://main--magnificent-licorice-5d2277.netlify.app/).

This is still a WIP as we are waiting for more libraries to support Wagmi v2

### Setup

Create an `.env` and add `VITE_WALLET_CONNECT_PROJECT_ID` to it. You can find it [here](https://cloud.walletconnect.com/app/project).
6 changes: 4 additions & 2 deletions packages/op-app/package.json
Original file line number Diff line number Diff line change
@@ -11,7 +11,8 @@
"typecheck": "tsc --noEmit --emitDeclarationOnly false",
"lint": "eslint \"**/*.{ts,tsx}\" && pnpm prettier --check \"**/*.{ts,tsx}\" --log-level=warn",
"lint:fix": "eslint \"**/*.{ts,tsx}\" --fix --quiet && pnpm prettier \"**/*.{ts,tsx}\" --write --log-level=warn",
"codegen": "ts-node --esm -T ./scripts/codegen"
"codegen": "ts-node --esm -T ./scripts/codegen",
"refresh": "vite build && vite preview"
},
"dependencies": {
"@eth-optimism/contracts-ts": "^0.17.0",
@@ -20,7 +21,8 @@
"prettier": "^3.1.0",
"react": "^18.2.0",
"react-dom": "^18.2.0",
"tsup": "^8.0.1"
"tsup": "^8.0.1",
"vite-plugin-react-refresh": "^0.3.0"
},
"devDependencies": {
"@eth-optimism/superchain-registry": "github:ethereum-optimism/superchain-registry#0496fcd6ee4e36f2bedd3e4fbffa9146b10219af",