Open askvortsov1 opened 3 years ago
This was initially proposed way back in https://github.com/flarum/core/issues/872#issuecomment-198682653. mixin seems to be unnecessary in recent years with ES6 and its class inheritance.
mixin
mixin would certainly need to be deprecated first, as it's currently used in 35 extensions (some deprecated) (https://query.flarum.dev/q/1bee4cae-d79a-44e2-adc8-5ce60579c2f2)
Example of a change (and relevant discussion) can be found here: https://github.com/flarum/core/pull/2351#issuecomment-705304663
This was initially proposed way back in https://github.com/flarum/core/issues/872#issuecomment-198682653.
mixinseems to be unnecessary in recent years with ES6 and its class inheritance.mixinwould certainly need to be deprecated first, as it's currently used in 35 extensions (some deprecated) (https://query.flarum.dev/q/1bee4cae-d79a-44e2-adc8-5ce60579c2f2)Example of a change (and relevant discussion) can be found here: https://github.com/flarum/core/pull/2351#issuecomment-705304663