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

[blockscout] apply celo theme #9

Closed

Conversation

diminator
Copy link

@diminator diminator commented Oct 1, 2019

Motivation

Re-brand blockscout to celo theme - fixes celo-org/celo-monorepo#997 and celo-org/celo-monorepo#994

Enhancements

blockscout_new_theme

Upgrading

prerequisite: #8
once that PR is merged - this PR can point to celo-org/blockscout:master

Env vars

export NETWORK=Celo
export NETWORK_ID=11001
export SUBNETWORK=<"Celo Integration" | "Celo Alfajores" | ...>
export ETHEREUM_JSONRPC_VARIANT=geth
export ETHEREUM_JSONRPC_HTTP_URL=http://localhost:8545
export ETHEREUM_JSONRPC_TRACE_URL=http://localhost:8545
export ETHEREUM_JSONRPC_WS_URL=ws://localhost:8546
export COIN=cGLD
export WS_SECRET=<WS_SECRET>
export POSTGRES_PASSWORD=<POSTGRES_SECRET>
export LOGO=/images/celo_logo.svg

@diminator diminator changed the title Feature/celo theme [blockscout] apply celo theme Oct 1, 2019
@tjlahey
Copy link

tjlahey commented Oct 4, 2019

This is great! Can I get a few screenshots of the entirety of the page, @diminator

If I have a few changes, I'd rather show than tell.

Do we have fonts that we can pick from?

@diminator
Copy link
Author

This is great! Can I get a few screenshots of the entirety of the page, @diminator

Here you can find some static screenshots:
celo-org/celo-monorepo#997 (comment)

Let me know if that works.

Do we have fonts that we can pick from?

Shouldn't be too hard to customize fonts. If you have some kit available we can switch over

@jcortejoso jcortejoso changed the base branch from master to feature/sync_upstream October 10, 2019 08:34
@jcortejoso jcortejoso requested a review from nambrot October 10, 2019 08:35
@jcortejoso
Copy link
Member

Replaced by #13

@jcortejoso jcortejoso closed this Oct 17, 2019
mrsmkl pushed a commit that referenced this pull request Dec 10, 2019
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.

3 participants