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

NEW Extend new AbstractGridFieldComponent class #367

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

GuySartorelli
Copy link
Collaborator

This makes the GridFieldQueuedJobExecute component Injectable, and allows any future enhancements in the new abstract class to automatically apply without requiring additional changes in this module.

The class is introduced in silverstripe/framework 4.11.0 (see silverstripe/silverstripe-framework#10204) so the dependency constraint needs to be updated.

@GuySartorelli
Copy link
Collaborator Author

Note that tests will error until a 4.11 branch or tag is available in silverstripe/silverstripe-framework

@GuySartorelli GuySartorelli force-pushed the enh/gridfieldcomponents-di branch from a4f5976 to b27b089 Compare February 12, 2022 07:19
@emteknetnz
Copy link
Collaborator

Could you please run git commit --amend --no-edit && git push -f to trigger a new build, this PR to travis-shared should ensure it now works with the ^4.11 framework requirement

@GuySartorelli GuySartorelli force-pushed the enh/gridfieldcomponents-di branch from b27b089 to c5d2b6e Compare March 3, 2022 04:58
@GuySartorelli
Copy link
Collaborator Author

@emteknetnz Done!

This makes the `GridFieldQueuedJobExecute` component `Injectable`, and allows any future enhancements in the new abstract class to automatically apply without requiring additional changes in this module.

The class is introduced in silverstripe/framework 4.11.0 so the dependency constraint needs to be updated.
@GuySartorelli GuySartorelli force-pushed the enh/gridfieldcomponents-di branch from c5d2b6e to d2e0488 Compare March 3, 2022 05:54
@GuySartorelli
Copy link
Collaborator Author

Fixed phpcs linting issue.

@emteknetnz emteknetnz merged commit 416d69e into symbiote:4 Mar 3, 2022
@GuySartorelli GuySartorelli deleted the enh/gridfieldcomponents-di branch March 3, 2022 21:46
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.

3 participants