This is fine, and I suppose it makes some sense to have the default admin app config be the base class - but in general I would go the other way and have the one that does less be the base class and the discovery one subclass and extend it. (Liskov something something mumble.)