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

feat: adjusted available extensions #4463

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

Conversation

Kemi-Elizabeth
Copy link
Collaborator

added note about adminpack below available extensions list

If there are any more extensions that don't work with later versions of postgresql, then perhaps we need to explore a table for which versions specifically work with which extensions.

Why

Closes #4462

What's changed

Added a note to https://docs.upsun.com/add-services/postgresql.html#available-extensions and https://docs.platform.sh/add-services/postgresql.html

Where are changes

Updates are for:

  • platform (sites/platform templates)
  • upsun (sites/upsun templates)

added note about adminpack below available extensions list
Copy link

@@ -666,11 +665,17 @@ extensions not listed here.

{{< note >}}

You can't upgrade to PostgreSQL 12 with the `postgis` extension enabled.
You can't upgrade to PostgreSQL 12 with the `postgis` extensions enabled.
Copy link
Collaborator

Choose a reason for hiding this comment

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

It is a singular extension, unless I'm missing something?

For more details, see how to [upgrade to PostgreSQL 12 with `postgis`](#upgrade-to-postgresql-12-with-the-postgis-extension).

{{< /note >}}

{{< note >}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I worry, given the length of the extensions section, that adding the note to the bottom will be completely missed by readers. When they get to the adminpack extension in the list, the note is still not in the viewport.

@@ -757,6 +757,13 @@ For more details, see how to [upgrade to PostgreSQL 12 with `postgis`](#upgrade-

{{< /note >}}

{{< note >}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I worry, given the length of the extensions section, that adding the note to the bottom will be completely missed by readers. When they get to the adminpack extension in the list, the note is still not in the viewport.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

adminpack extension no longer supported - postgresql update
3 participants