Skip to content

FIX ModelAdmin importer use with custom routes#1208

Closed
chillu wants to merge 1 commit intosilverstripe:1from
open-sausages:pulls/1/modeladmin-importers
Closed

FIX ModelAdmin importer use with custom routes#1208
chillu wants to merge 1 commit intosilverstripe:1from
open-sausages:pulls/1/modeladmin-importers

Conversation

@chillu
Copy link
Member

@chillu chillu commented May 13, 2021

Regression from when we introduced $modelTab.
This is backwards compatible since $modelTab falls back to $modelClass.

Also improved $model_importers to fall back to the default per model,
rather than requiring definition of all models on a non-null value.

Regression from when we introduced $modelTab.
This is backwards compatible since $modelTab falls back to $modelClass.

Also improved $model_importers to fall back to the default per model,
rather than requiring definition of *all* models on a non-null value.
@chillu
Copy link
Member Author

chillu commented May 21, 2021

Superseded by #1216

@chillu chillu closed this May 21, 2021
@chillu chillu deleted the pulls/1/modeladmin-importers branch May 21, 2021 04:04
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