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.

Sign up for GitHub

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

Jump to bottom

FIX Don't require classes that don't exist #195

Merged

emteknetnz merged 1 commit into silverstripe:1 from creative-commoners:pulls/1/dont-require-elemental

Jul 28, 2024

Member

GuySartorelli commented Jul 28, 2024

Follow on from #194

If the extension doesn't exist, the model causes errors. If the model doesn't exist the admin probably will too.
See https://github.com/silverstripe/silverstripe-admin/actions/runs/10135520139/job/28023093972

Issue

Broken builds and merge-ups .github#287


          FIX Don't require classes that don't exist

GuySartorelli mentioned this pull request

Broken builds and merge-ups silverstripe/.github#287

Closed

emteknetnz approved these changes

View reviewed changes

emteknetnz merged commit 579e160 into silverstripe:1

1 check passed

emteknetnz deleted the pulls/1/dont-require-elemental branch

July 28, 2024 23:45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet