Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Setup dApp skeleton #7

Closed
wants to merge 23 commits into from
Closed
Show file tree
Hide file tree
Changes from 5 commits
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
4 changes: 4 additions & 0 deletions .babelrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
{
"presets": ["next/babel"],
"plugins": [["styled-components", { "ssr": true }]]
}
14 changes: 14 additions & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
version: 2.1

executors:
docker-node:
docker:
- image: circleci/node:11

jobs:
test_solidity:
executor: docker-node
steps:
- checkout
- run: npm i
- run: npm run build
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if this will work. Maybe we should drop the circleci configuration from this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha could you be less specific? Why won't it work?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both of you could use some details heh. Based on the commit that introduced this, I have no idea what you're trying to achieve. What's the purpose of adding this? That will probably help Kuba understand what is expected to work here.

It certainly doesn't test solidity, which is what your build currently implies 😉

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha could you be less specific? Why won't it work?

Yes, sorry. This will be executed in the root directory, which doesn't contain these commands definition. We should set working_directory for these steps.
We should also add workflows specification at the end of the file.

If we want to configure it all in this PR I can give you a hand with it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closed but responding anyways - good catch guys, this was actually because I moved everything into app/ to play well with #8's file layout. ATT of that commit @Shadowfiend, it was correct and simply did a build.

3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
node_modules
yarn.lock
.next
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
# tbtc-dapp
# tbtc-dapp

21 changes: 21 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
{
"name": "tbtc-dapp",
"version": "0.1.0",
"scripts": {
"dev": "next",
"build": "next build",
"start": "next start"
},
"dependencies": {
"babel-plugin-styled-components": "^1.10.6",
"next": "^9.0.2",
"normalize.css": "^8.0.1",
"react": "^16.8.6",
"react-dom": "^16.8.6",
"react-redux": "^7.1.0",
"redux": "^4.0.4",
"redux-devtools-extension": "^2.13.8",
"styled-components": "^4.3.2",
"web3": "^1.2.0"
}
}
5 changes: 5 additions & 0 deletions pages/index.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
const Index = () => {
return <div>Hello tBTC!</div>
}

export default Index