Skip to content

Conversation

@rdig
Copy link
Member

@rdig rdig commented Jun 22, 2020

This PR is a small change that will be pushed into production alongside the new v2.5.0 app release that enables browsing the app w/o first connection a wallet.

In order to take advantage of that, we won't direct users from the website to the /connect route anymore, but instead, prompt them to go browse the Betacolony.

Changes

  • Header now points to directly to the beta colony

Screenshots

Screenshot from 2020-06-22 17-25-00
Screenshot from 2020-06-22 17-25-13

@rdig rdig requested review from ceolson01 and chmanie June 22, 2020 14:39
@rdig rdig self-assigned this Jun 22, 2020
Copy link
Contributor

@ceolson01 ceolson01 left a comment

Choose a reason for hiding this comment

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

Nice 'n' easy! 👏

Copy link
Member

@chmanie chmanie left a comment

Choose a reason for hiding this comment

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

I don't know why CI is failing but this seems fine!

@rdig
Copy link
Member Author

rdig commented Jun 24, 2020

CI is still on Circle and I believe we removed this project as well but forgot to update the build pipeline in here. I think?

@chmanie
Copy link
Member

chmanie commented Jun 24, 2020

It should be using circle. It's actually used to deploy it I think so it has to work. I'll investigate

@chmanie
Copy link
Member

chmanie commented Jun 24, 2020

Back on!

@rdig rdig merged commit 8309aa9 into generator Jun 24, 2020
@rdig rdig deleted the fix/change-link-to-app branch June 24, 2020 15:27
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.

4 participants