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

API Make CMSMain more generic #3017

Conversation

GuySartorelli
Copy link
Member

@GuySartorelli GuySartorelli commented Oct 16, 2024

Remove hardcoded references to pages and SiteTree
Remove assumption that records are versioned
Remove or validate assumptions about methods on the model class
Improve general architecture of CMSMain

Relies on changes in silverstripe/silverstripe-admin#1837

Issue

@GuySartorelli GuySartorelli marked this pull request as draft October 16, 2024 23:52
@GuySartorelli GuySartorelli force-pushed the pulls/6/generic-cmsmain branch 7 times, most recently from 2faf391 to db3dcf7 Compare October 22, 2024 02:32
@GuySartorelli GuySartorelli force-pushed the pulls/6/generic-cmsmain branch 4 times, most recently from 90b096b to da66f41 Compare November 7, 2024 03:15
@GuySartorelli GuySartorelli force-pushed the pulls/6/generic-cmsmain branch 6 times, most recently from 9475eee to 05eb821 Compare November 8, 2024 01:18
@GuySartorelli GuySartorelli force-pushed the pulls/6/generic-cmsmain branch 3 times, most recently from d758dba to 5bcb887 Compare November 8, 2024 01:43
@GuySartorelli GuySartorelli force-pushed the pulls/6/generic-cmsmain branch from 5bcb887 to cf093d1 Compare November 14, 2024 00:53
@GuySartorelli GuySartorelli force-pushed the pulls/6/generic-cmsmain branch 7 times, most recently from 3cca2a1 to 32bca92 Compare December 3, 2024 23:39
Remove hardcoded references to pages and SiteTree
Remove assumption that records are versioned
Remove or validate assumptions about methods on the model class
Improve general architecture of CMSMain
@GuySartorelli
Copy link
Member Author

Replaced with #3035

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.

1 participant