Skip to content

Conversation

@sadiqkhoja
Copy link
Contributor

@sadiqkhoja sadiqkhoja commented Jan 8, 2026

Closes #1556

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

  • All tests are passing
  • Following manual verification done after upgrade:
    • Existing draft form works
    • Existing published Form works
    • Edit existing submissions work
    • Existing public link works
    • Existing single submission public link works
    • New form can be created and published
    • New submission can be made to the new Form
    • New submission is editable

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

NA

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

Enketo express has quite an old redis client i.e. v3.1.2 which may not be fully compatible with the redis v8, but the usage is quite basic so it should be okay to upgrade.

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

No

Before submitting this PR, please make sure you have:

  • branched off and targeted the next branch OR only changed documentation/infrastructure (master is stable and used in production)
  • verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced

Copy link
Member

@matthew-white matthew-white 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 not an expert on redis, but the reasoning in the PR description makes sense to me. Let's get this merged and see how it does on staging. 👍

@matthew-white matthew-white linked an issue Feb 5, 2026 that may be closed by this pull request
@sadiqkhoja sadiqkhoja merged commit 91bfcdb into getodk:next Feb 5, 2026
9 of 10 checks passed
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.

Upgrade redis to v8

2 participants