Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Sparrow dom/rewards active campaign #4263

Merged
merged 20 commits into from
Jan 13, 2020
Merged

Conversation

sparrowDom
Copy link
Member

@sparrowDom sparrowDom commented Jan 13, 2020

  • display a message when there is not active rewards campaign
  • fix docker environment setup

Fixes: #4259

Checklist:

@@ -210,7 +210,7 @@ services:
command:
>
/bin/bash -c "wait-for.sh -t 0 -q elasticsearch:9200 --
wait-for.sh -t 0 -q services:1111 --
wait-for.sh -t 0 -q services:8545 --
Copy link
Member Author

Choose a reason for hiding this comment

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

@mikeshultz have changed this back from port 1111 to 8454. It was preventing some of the services in docker to start up.

Copy link
Contributor

@mikeshultz mikeshultz Jan 13, 2020

Choose a reason for hiding this comment

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

Perhaps because something with services isn't working? This is preventing other services from coming up because some dependency services (probably) isn't coming up. It specifically brings up an empty service on port 1111 to indicate that everything is good to go:

net.createServer().listen(1111, '0.0.0.0')

By checking 8545(ganache), you're skipping basically every other service that it brings up. I'd recommend looking into the cause of your issue instead of making this change.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah thanks @mikeshultz The problem was with identity contract not deploying. Fix here:
#4274

@sparrowDom
Copy link
Member Author

@micahalcorn @franckc currently I have it set to this:
Screenshot 2020-01-13 15 31 53

Happy to change it if needed.

@micahalcorn
Copy link
Member

micahalcorn commented Jan 13, 2020

Thanks @sparrowDom 👍 Should we link to the blog post there announcing the early end?

@sparrowDom
Copy link
Member Author

@micahalcorn was thinking about linking to the blog post, but we will show this screen anytime where there is no active campaigns. The block post link might be outdated with the next occurrence when we close a campaign early.

@franckc
Copy link
Contributor

franckc commented Jan 13, 2020

Chatted with @joshfraser - he recommends we put a link to the blogpost and to our telegram english channel. We'll have to update the blog post link in the future... but this is still worth doing to in order to minimize the amount of users inquiries in the short-term.

@franckc
Copy link
Contributor

franckc commented Jan 13, 2020

@micahalcorn Could you please check the copies before I merge this in? Thanks:)

@franckc franckc merged commit 64e09c4 into master Jan 13, 2020
@franckc franckc deleted the sparrowDom/rewardsActiveCampaign branch January 13, 2020 22:06
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.

Campaigns home page crash when there is no active campaign
5 participants