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

Setup truffle suite with drizzle on the front end #24

Merged
merged 1 commit into from
Oct 22, 2018

Conversation

barlock
Copy link
Contributor

@barlock barlock commented Oct 18, 2018

Related Issue
Supports #3

Related PRs
This PR is not dependent on any other PR

What does this PR do?
Adds truffle and drizzle as a POC. All this does is uses ganache-cli to fake-deploy a contract and give you an address back.

Next step is to get uPort working with drizzle. This pr was getting out of hand so I wanted to get it commited sooner rather than later.

Description of Changes
trufflesuite/drizzle#130

drizzle+react native is where a lot of the time/effort/pain is. Lots of shimming all detailed in that issue. The rest is straight forward. I'll write some tests for the drizzle provider immediately next. If you'd like to see those in this pr we can wait, but the shimming was getting big.

What gif most accurately describes how I feel towards this PR?
devastated

@barlock barlock force-pushed the truffles-everywhere branch from 802b0a5 to 25e429a Compare October 18, 2018 21:22
@@ -0,0 +1,16 @@
module.exports = api => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Jest, babel and react-native don't like each other a lot. facebook/react-native#21075

@@ -1,61 +1,13 @@
import React, { Component } from 'react';
import { Platform, StyleSheet, Text, View, Button } from 'react-native';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove display code to put it into a /screen so that this is an initialization component.

@barlock barlock force-pushed the truffles-everywhere branch from 25e429a to bd4ac4f Compare October 19, 2018 13:34
@barlock barlock force-pushed the truffles-everywhere branch from bd4ac4f to 3911024 Compare October 19, 2018 13:45
@hadasz hadasz merged commit 01e8e0c into master Oct 22, 2018
@barlock barlock deleted the truffles-everywhere branch October 25, 2018 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants