-
Notifications
You must be signed in to change notification settings - Fork 52
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
Proposed future for module (SS4.x and beyond) #209
Comments
@chillu Has there been any update to this internally or a plan around resourcing? |
There are no plans to make this a supported module for SilverStripe 4.x at the moment, but we're happy to provide feedback on community-driven pull requests for 4.x compatibility :) |
We are having a look at this now. |
We have upgraded the module to SS4 (using an AUTOMATED upgrader ONLY): https://github.com/sunnysideup/silverstripe-dms so that we can cherry pick from it if needed. |
That's great @sunnysideup - have you been able to fully feature test it? |
@NightJar - definitely NOT. We just did an automated upgrade only! |
Is there any progress on this? It's widely requested to be ported, it's blocking multiple agencies from upgrading :'( |
This module has been a stop-gap solution for SS 3.x while we improved the core asset (and document) management capabilities in SS 4.x. We now have a lot more capabilities built-in there, which begs the question what happens with this module. Here's some starters for a discussion.
Built-in to SS 4.x (and CWP 2.x):
silverstripe/fulltextsearch
- part of CWP recipe)silverstripe/textextraction
- part of CWP recipe)Worth retaining as module features:
Worth adding as module features:
Suggest to remove:
Document
hasFile
, after:File
can be a document)Notes:
Document
toFile
model, we'll need to provide a database migration path. It would also mean non-trivial migration for anyone who implemented customDocument
implementations, although I doubt anyone has done that.The text was updated successfully, but these errors were encountered: