Skip to content

Conversation

@dkastl
Copy link
Member

@dkastl dkastl commented Jun 2, 2024

Fixes #29

Not sure if this can be updated like this or requires a database migration for existing installations as the setting value seems to be stored with the wrong spelling also in the database.

@sanak
Copy link
Member

sanak commented Jun 2, 2024

@dkastl Thanks!

And yes, migration process is necessary, as I mentioned in #27 description.

@sanak
Copy link
Member

sanak commented May 1, 2025

@dkastl Thanks for creating this PR. I am thinking to do the followings on this PR branch, so just for your information.

  • Rebase from the latest next branch and force push
  • Add expended => expanded migration
  • Drop Redmine <= 5.0 support
    • Some part can be simplified
  • Add CI badge

Fixes #29

Signed-off-by: Daniel Kastl <daniel@georepublic.de>
@sanak sanak force-pushed the dkastl/issue29 branch from 1b9bab3 to 2bf3da7 Compare May 1, 2025 08:42
@sanak sanak requested review from mopinfish and sanak May 2, 2025 04:53
@sanak sanak self-assigned this May 2, 2025
@sanak
Copy link
Member

sanak commented May 2, 2025

@dkastl @mopinfish
I made the following changes, so please check it after holidays.
Additional migration should be fine when old all_expended is used in settings and user_preferences tables.
2bf3da7...dkastl/issue29/

@sanak
Copy link
Member

sanak commented May 7, 2025

Well, I will merge this once.

@sanak sanak merged commit 83219cc into next May 7, 2025
21 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.

3 participants