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

@sanak sanak force-pushed the dkastl/issue29 branch from 1b9bab3 to 2bf3da7 Compare May 1, 2025 08:42
@sanak sanak requested review from sanak and mopinfish 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.

2 participants