Skip to content

upgrade-postgres: clarify data cleanup link - #2215

Merged
alxndrsn merged 2 commits into
getodk:nextfrom
alxndrsn:postgres-upgrade-instructions-link
Sep 17, 2026
Merged

alxndrsn merged 2 commits into
getodk:nextfrom
alxndrsn:postgres-upgrade-instructions-link

Conversation

@alxndrsn

@alxndrsn alxndrsn commented Sep 7, 2026 •

Copy link
Copy Markdown
Contributor

In the original implementation (#343), old postgres data was cleaned up automatically if a specific flag file was present.

This behaviour was changed (prior to release?) in a follow-up PR (#395) to link to documentation instead.

Over time, it's become less clear which part of the linked documentation is relevant. This PR aims to link more directly to the relevant documentation.

What has been done to verify that this works as intended?

  • clicked the link
  • ci
  • not much else

Why is this the best possible solution? Were any other approaches considered?

It took me a while to understand which part of the original link (https://docs.getodk.org/central-upgrade/) was relevant to upgrading postgres. The new link is https://docs.getodk.org/central-upgrade/#upgrading-to-central-v2023-2

How does this change impact users? Describe intentional behavior changes from code updates. What are the regression risks?

Hopefully gets them where they need to be quicker. Low risk of regression.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

In the original implementation (getodk#343), old postgres data was cleaned up automatically if a specific flag file was present.

This behaviour was changed (prior to release?) in a follow-up PR (getodk#395) to link to documentation instead.

Over time, it's become less clear which part of the linked documentation is relevant.  This PR aims to link more directly to the relevant documentation.
Comment thread files/postgres/upgrade-postgres.sh Outdated
@alxndrsn
alxndrsn marked this pull request as ready for review September 7, 2026 10:21
@alxndrsn
alxndrsn requested a review from lognaturel September 7, 2026 10:21
@alxndrsn
alxndrsn merged commit e83be33 into getodk:next Sep 17, 2026
7 checks passed
@alxndrsn
alxndrsn deleted the postgres-upgrade-instructions-link branch September 17, 2026 08:21
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