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

Finance: refactor to new styles for 0.8 #929

Merged
merged 107 commits into from
Aug 19, 2019
Merged

Finance: refactor to new styles for 0.8 #929

merged 107 commits into from
Aug 19, 2019

Conversation

AquiGorka
Copy link
Contributor

@AquiGorka AquiGorka commented Jul 18, 2019

image

How to test

Assuming everything is in the ~/projects directory (replace by your own).

First time

Clone this repo + Aragon client’s + aragonUI’s if not done already:

cd ~/projects
git clone git@github.com:aragon/aragon-apps.git
git clone git@github.com:aragon/aragon.git
git clone git@github.com:aragon/aragon-ui.git

Create a link for aragonUI (npm might tell you that it is already done):

cd ~/projects/aragon-ui
npm link

Getting updates

Here we are updating the branches + reinstalling the dependencies + linking aragonUI again.

cd ~/projects/aragon-ui
git checkout newstyle
git pull origin newstyle
npm install

cd ~/projects/aragon
git checkout newstyle
git pull origin newstyle
npm install
npm link @aragon/ui

cd ~/projects/aragon-apps/apps/finance/app
git checkout newstyle/finance
git pull origin newstyle/finance
npm install
npm link @aragon/ui

Running it

We can now start the two servers (the app + Aragon client). In one shell session (or tab in your terminal app):

cd ~/projects/aragon-apps/apps/finance/app
npm start

In another session (we are asking the client to load apps running locally):

cd ~/projects/aragon
env REACT_APP_ASSET_BRIDGE=local yarn start

Open http://localhost:3000/ in your browser.

@AquiGorka AquiGorka requested review from bpierre, 2color and sohkai July 18, 2019 13:32
@dizzypaty dizzypaty self-requested a review July 18, 2019 14:19
@coveralls
Copy link

coveralls commented Jul 18, 2019

Coverage Status

Coverage remained the same at 97.914% when pulling 25fac75 on newstyle/finance into a4f873e on newstyle-0.8.

@sohkai
Copy link
Contributor

sohkai commented Aug 17, 2019

@dizzypaty I've lowered the vertical padding of the empty state just a tad: the designs have them at around 160px (20 grid units) but I've taken them down to 120px (15 grid units) because on a 1440x900 display (usually a 13' laptop), the text gets cut off:

Screen Shot 2019-08-18 at 1 02 22 AM

@sohkai sohkai merged commit 2e8e1f3 into newstyle-0.8 Aug 19, 2019
@sohkai sohkai deleted the newstyle/finance branch August 19, 2019 10:50
sohkai pushed a commit that referenced this pull request Sep 2, 2019
facuspagnuolo pushed a commit that referenced this pull request Sep 3, 2019
ramilexe pushed a commit to ConsiderItDone/aragon-apps that referenced this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants