-
Notifications
You must be signed in to change notification settings - Fork 509
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
chore: Remove old integration pages #17958
chore: Remove old integration pages #17958
Conversation
@@ -86,7 +86,7 @@ export const ALL_PLUGINS: Plugin[] = [ | |||
logoDark: "/images/logos/plugins/firestore-dark.svg", | |||
id: "firestore", | |||
kind: "source", | |||
availability: "free", | |||
availability: "premium", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's more cleanup to be done to remove any plugin data from the old Website, but didn't want to dig too deep into it
@@ -870,114 +870,6 @@ | |||
"source": "/docs/plugins/destinations/sqlite/:path*", | |||
"destination": "https://hub.cloudquery.io/plugins/destination/cloudquery/sqlite" | |||
}, | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have to remove the existing redirects otherwise https://github.com/cloudquery/cloudquery/pull/16681/files#diff-21378a71578c160a16aa86a2ccc6120c09d1d10a3deaf16af651a228bd0cfde4R1054 won't match
7011a2e
to
41ffbfa
Compare
41ffbfa
to
9a3716f
Compare
9a3716f
to
7508e52
Compare
Summary
Not sure why the Website build didn't fail in #17903, but it fails on
main
:We shouldn't generate integration pages at all and redirect per #16681