Skip to content
This repository has been archived by the owner on Feb 11, 2019. It is now read-only.

5-Step Launch Wizard #91

Merged
merged 11 commits into from
Dec 18, 2018
Merged

5-Step Launch Wizard #91

merged 11 commits into from
Dec 18, 2018

Conversation

acrdlph
Copy link
Member

@acrdlph acrdlph commented Dec 17, 2018

just a few loose ends left here: help popovers are missing, review step only shows 1 service and prompt texts are not perfect yet.

@acrdlph acrdlph changed the title 5-Step Launch Wizard [WIP] 5-Step Launch Wizard Dec 17, 2018
@acrdlph acrdlph changed the title [WIP] 5-Step Launch Wizard 5-Step Launch Wizard Dec 17, 2018
@acrdlph
Copy link
Member Author

acrdlph commented Dec 17, 2018

I think its ready! Hope I gitignored the right thing..

@acrdlph
Copy link
Member Author

acrdlph commented Dec 17, 2018

took the fix note for #82 out again, since i will still have to do some work on that

@acrdlph acrdlph requested a review from lsaether December 18, 2018 08:41
Copy link
Contributor

@lsaether lsaether left a comment

Choose a reason for hiding this comment

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

The buidl files are still not right :( They need to have network 4 for Rinkeby with the same addresses and transaction hashes as found in the build files on master

@lsaether
Copy link
Contributor

@acrdlph
Copy link
Member Author

acrdlph commented Dec 18, 2018

something weird is going on here.. the build files in the pushed repo are different from the ones i have locally ?!

@lsaether
Copy link
Contributor

You copied the PersonalEconomyFactory to PersonalEconomy artifact file xD

Also I'm in appear if you want to join

@acrdlph
Copy link
Member Author

acrdlph commented Dec 18, 2018

argh yea got it now!

@lsaether lsaether merged commit 67d1546 into master Dec 18, 2018
@lsaether lsaether deleted the StepWizard branch December 18, 2018 12:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants