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

Fix/show only mainnet bounties #7637

Merged
merged 16 commits into from
Nov 11, 2020

Conversation

chibie
Copy link
Contributor

@chibie chibie commented Oct 7, 2020

Description

Follow up on this #7584

This resolves ValueError invalid literal for int() with base 10: '' noticed in prod

https://www.loom.com/share/35eab9652d2b440b8fe3729c0d940eb6

Refers/Fixes

https://sentry.io/organizations/gitcoin/issues/1941745874/?referrer=slack

# Bounty QuerySet Manager
objects = BountyQuerySet.as_manager()
# Bounty Manager from QuerySet
objects = BountyManager.from_queryset(BountyQuerySet)()
Copy link
Contributor

Choose a reason for hiding this comment

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

how this will impact the ability to filter by rinkeby bounties on the explorer?
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@octavioamu oops... should i take off rinkeby filter in production, since we only want to deal with mainnet in production?

Copy link
Contributor

Choose a reason for hiding this comment

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

still need to be able to filter by network in some places, is just the post on the townsquare need to be avoided on prod

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@octavioamu all set now. i've excluded explorer

@octavioamu octavioamu merged commit 3398d08 into gitcoinco:master Nov 11, 2020
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