Skip to content
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

DOC Modules loosing commercial support #633

Open
wants to merge 1 commit into
base: 6
Choose a base branch
from

Conversation

emteknetnz
Copy link
Member

@emteknetnz emteknetnz commented Nov 19, 2024

@emteknetnz emteknetnz marked this pull request as draft November 19, 2024 22:30
@emteknetnz emteknetnz force-pushed the pulls/6/remove-support branch 12 times, most recently from 5cd02f4 to 0d8aa16 Compare November 25, 2024 01:33
@emteknetnz emteknetnz force-pushed the pulls/6/remove-support branch 2 times, most recently from 6c2bc70 to bc5f547 Compare November 26, 2024 03:01
@emteknetnz emteknetnz marked this pull request as ready for review November 26, 2024 04:32
@@ -791,6 +793,26 @@ Note that the change from `ViewableData` to `ModelData` specifically was made to
|`SilverStripe\View\ViewableData_Customised`|[`SilverStripe\Model\ModelDataCustomised`](api:SilverStripe\Model\ModelDataCustomised)|
|`SilverStripe\View\ViewableData_Debugger`|[`SilverStripe\Model\ModelDataDebugger`](api:SilverStripe\Model\ModelDataDebugger)|

The following classes have been changed when code was move to the `silverstripe/reports` module from one of the modules that lost commercial support:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The following classes have been changed when code was move to the `silverstripe/reports` module from one of the modules that lost commercial support:
The following classes have been changed when code was move to the `silverstripe/reports` module from some of the modules that lost commercial support:

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated


The `silverstripe/campaign-admin` was a core module that provided a way to publish multiple related records at the same time, but it was very rarely used in practice. It has been removed from the default CMS installation and has had its commercial support removed. Its integration support has been removed from `silverstripe/cms`, `silverstripe/admin`, and `silverstripe/asset-admin`, all of which provided "Add to campaign" UX functionality. This means that it will not be possible to get any sort of campaign admin functionality in CMS 6 even if using a forked version of `silverstripe/campaign-admin`.

### Modules loosing commercial support {#modules-loosing-support}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this to be at the top, to be consistent with the CMS 5 changelog.

Give it the same headings as we used in CMS (with the obvious exclusion of "Modules with upgrade guides" since we haven't done any). Move the "Campaign admin removed from the CMS" section to be under "Change to supported modules"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done


A number of [commercially supported modules](https://docs.silverstripe.org/en/project_governance/supported_modules/) receive ongoing releases and maintainance. With the release of Silverstripe CMS 6.0.0, the following modules will no longer be commercially supported:

- bringyourownideas/silverstripe-composer-update-checker
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Structure this the same was as we did for CMS 5.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done


### Modules loosing commercial support {#modules-loosing-support}

A number of [commercially supported modules](https://docs.silverstripe.org/en/project_governance/supported_modules/) receive ongoing releases and maintainance. With the release of Silverstripe CMS 6.0.0, the following modules will no longer be commercially supported:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the same or similar wording as we used in CMS 5.

Copy link
Member Author

@emteknetnz emteknetnz Nov 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

[dnadesign/silverstripe-elemental](https://packagist.org/packages/dnadesign/silverstripe-elemental) | 5
[dnadesign/silverstripe-elemental-userforms](https://packagist.org/packages/dnadesign/silverstripe-elemental-userforms) | 4
[silverstripe-themes/simple](https://packagist.org/packages/silverstripe-themes/simple) | 3
[silverstripe/auditor](https://packagist.org/packages/silverstripe/auditor) | 3
[silverstripe/blog](https://packagist.org/packages/silverstripe/blog) | 4
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[silverstripe/blog](https://packagist.org/packages/silverstripe/blog) | 4

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

[silverstripe/segment-field](https://packagist.org/packages/silverstripe/segment-field) | 3
[silverstripe/sharedraftcontent](https://packagist.org/packages/silverstripe/sharedraftcontent) | 3
[silverstripe/sitewidecontent-report](https://packagist.org/packages/silverstripe/sitewidecontent-report) | 4
[silverstripe/spamprotection](https://packagist.org/packages/silverstripe/spamprotection) | 4
[silverstripe/staticpublishqueue](https://packagist.org/packages/silverstripe/staticpublishqueue) | 6
[silverstripe/subsites](https://packagist.org/packages/silverstripe/subsites) | 3
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[silverstripe/subsites](https://packagist.org/packages/silverstripe/subsites) | 3

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

@emteknetnz emteknetnz force-pushed the pulls/6/remove-support branch 3 times, most recently from 6ecc999 to 01f51f2 Compare November 27, 2024 01:13
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