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

Remove documented support for Postgres #3227

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

mediaformat
Copy link
Contributor

@mediaformat mediaformat commented May 5, 2024

Here's what I fixed or added:

Remove documented support for Postgres

Here's why I did it:

Postgres support was removed in 737502f

Checklist:

  • This pull request addresses a single issue
  • My git branch is named in a descriptive way - i.e., yourname-summary-of-issue

@mediaformat mediaformat requested a review from benwerd as a code owner May 5, 2024 20:44
@benwerd
Copy link
Member

benwerd commented May 5, 2024

Aiieeeee.

I'm going to put this PR on hold for a second and try to bring back Postgres support this week. This was removed while I wasn't watching the project and I deeply regret it.

@mediaformat
Copy link
Contributor Author

@benwerd If no one has been using Postgres with Known in the last 4 years, should we still support postgres migration in ActivityPub/Sprint 2? is this a blocker?

@benwerd
Copy link
Member

benwerd commented May 5, 2024

@mediaformat Not a blocker for the ActivityPub project at all! I just want to bring it back online in the wider context. I guess we can remove it from the documentation for now and then I'll reinstate it when it's ready.

Copy link
Member

@benwerd benwerd left a comment

Choose a reason for hiding this comment

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

I'm sad that this happened, but this PR brings the documentation into truthfulness. LG!

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.

Postgres support status
2 participants