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

Dev blocked projects and block Metakeys copy-cat #4067

Merged
merged 2 commits into from
Sep 14, 2023

Conversation

johnnyd-eth
Copy link
Contributor

@johnnyd-eth johnnyd-eth commented Sep 14, 2023

For projects listed in BLOCKED_PROJECT_IDS:

  • Hides from project listings
  • Shows banner on project page
  • Disables pay button
Screen Shot 2023-09-14 at 2 31 27 pm

@vercel
Copy link

vercel bot commented Sep 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Updated (UTC)
juice-interface ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 5:18am
juice-interface-goerli ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 5:18am

@@ -68,7 +69,13 @@ export function useDBProjectsInfiniteQuery(
pageSize,
})}`,
)
.then(res => (res.data ? res.data.map(parseDBProject) : []))
.then(res =>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adds filters to "all" (same as the "new" query) and "trending" projects queries

@@ -0,0 +1,4 @@
// List of delisted projects
export const BLOCKED_PROJECT_IDS = [
'564', //copycat of 563
Copy link
Contributor

@peripheralist peripheralist Sep 14, 2023

Choose a reason for hiding this comment

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

we'll want these to be project ids (e.g. "2-564"), not projectIds (otherwise this would block a v1 project 564). we also want it to be specific to mainnet

Copy link
Contributor

@peripheralist peripheralist Sep 14, 2023

Choose a reason for hiding this comment

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

or, we could copy the pattern used for archived projects (storing V2ArchivedProjectIdsByNetwork that are all pv == 2), but we need to add the pv == 2 statement in the filter. archived project lists are network-specific

Copy link
Contributor

Choose a reason for hiding this comment

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

image image

cleanup project filtering on db projects queries
@peripheralist peripheralist merged commit 326ffb7 into main Sep 14, 2023
4 checks passed
@peripheralist peripheralist deleted the delist-copy-project branch September 14, 2023 05:20
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.

2 participants